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.

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

Closed
wants to merge 6 commits into from
Closed

Java 8 upgrade #83

wants to merge 6 commits into from

Conversation

vvagaytsev
Copy link

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
Copy link

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.

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants