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

Removed ConcurrentHashMapV8 #7392

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
4 participants
@cfontes
Copy link
Contributor

commented Aug 22, 2014

Basically reverting #6400 as requested by @mikemccand

Closes #7296

@mikemccand

This comment has been minimized.

Copy link
Contributor

commented Aug 26, 2014

Thank you @cfontes I see your CLA on file ... I'll have a look here.

@mikemccand

This comment has been minimized.

Copy link
Contributor

commented Aug 26, 2014

OK this LGTM, I'll push shortly. Thank you @cfontes!

mikemccand added a commit that referenced this pull request Aug 26, 2014

Core: use Java's built-in ConcurrentHashMap
It's risky to have our own snapshot of Java 8's ConcurrentHashMap:
unless we keep the sources in sync over time (and OpenJDK's version
had already diverged), then we won't get bug/performance fixes.  Users
can choose to upgrade to Java 8 to see the improvements of CHM.

Closes #7392

Closes #7296

@mikemccand mikemccand removed review labels Aug 26, 2014

@cfontes

This comment has been minimized.

Copy link
Contributor Author

commented Aug 27, 2014

Thank you! ES is awesome, feels great to help.

mikemccand added a commit that referenced this pull request Sep 8, 2014

Core: use Java's built-in ConcurrentHashMap
It's risky to have our own snapshot of Java 8's ConcurrentHashMap:
unless we keep the sources in sync over time (and OpenJDK's version
had already diverged), then we won't get bug/performance fixes.  Users
can choose to upgrade to Java 8 to see the improvements of CHM.

Closes #7392

Closes #7296

@clintongormley clintongormley changed the title removing JSR-166e ConcurrentHashMap Internal: Removed ConcurrentHashMapV8 Sep 11, 2014

@clintongormley clintongormley changed the title Internal: Removed ConcurrentHashMapV8 Removed ConcurrentHashMapV8 Jun 7, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.