Skip to content
Please note that GitHub no longer supports your web browser.

We recommend upgrading to the latest Google Chrome or Firefox.

Learn more
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.

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

Java 8 upgrade #83

Open
wants to merge 6 commits into
base: master
from
Open

Java 8 upgrade #83

wants to merge 6 commits into from

Conversation

@vvagaytsev
Copy link

vvagaytsev commented Jan 14, 2018

Covers the issue #76 . It can be maintained in a separate branch under the different artifact name (like language-detector-java-8) if the compatibility with the deprecated Java versions matters.

The commits include some language level changes as well. There are still some code places where Java 8 Stream API can be used. I can chnage those too.

Feel free to contact me and give any feedback.

Regards,
Vladimir

@seinecle

This comment has been minimized.

Copy link

seinecle commented Jan 12, 2020

Two remarks on the Java 8 version:

  • would be nice if it would enable to drop the dependency to Guava
  • streams or parallel streams? Parallel streams have some overheads, which are not worth it when the task is particularly short.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.