Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Support Java 8 Optional #21

Open
whiskeysierra opened this issue Jun 19, 2015 · 3 comments
Open

Support Java 8 Optional #21

whiskeysierra opened this issue Jun 19, 2015 · 3 comments

Comments

@whiskeysierra
Copy link
Contributor

Right now there is explicit support for Guava's Optional (see here. It would be nice to support the Java 8 version as well. Hardcoding it just like the other is probably a bad idea as it would require to drop Java 7 support, so we most likely need some plugin mechanisms there, e.g. ServiceLoader?
@zd0
Copy link

zd0 commented May 25, 2016

+1 for Java 8's Optional support. Personally, doesn't matter if it would be hardcoded and Java 7 is dropped.
Currently, the code that has switched to Java 8 Optional and still uses this library has ugly transformations scattered around.

zd0 pushed a commit to zd0/java-sproc-wrapper that referenced this issue May 27, 2016
Support Java 8 Optional (through reflections) zalando-stups#21
ecc3780
@valgog
Copy link
Collaborator

valgog commented Aug 17, 2016

@zd0 does this imply that we are loosing Java 7 compatibility with this change?

@whiskeysierra
Copy link
Contributor Author

No, it shouldn't

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@whiskeysierra @valgog @zd0