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

Migrate from Trove to Hppc #3858

Closed
martijnvg opened this Issue Oct 9, 2013 · 5 comments

Comments

Projects
None yet
2 participants
@martijnvg
Copy link
Member

martijnvg commented Oct 9, 2013

No description provided.

@ghost ghost assigned martijnvg Oct 9, 2013

@martijnvg

This comment has been minimized.

Copy link
Member Author

martijnvg commented Oct 9, 2013

Fixed via 088e05b

@martijnvg martijnvg closed this Oct 9, 2013

@karussell

This comment has been minimized.

Copy link
Contributor

karussell commented Oct 10, 2013

@martijnvg why was the move done? license? performance? ease of use? Or because of stability issues?

http://elasticsearch-users.115913.n3.nabble.com/Anyone-have-G1-GC-working-What-environment-configs-td4039878.html

@martijnvg

This comment has been minimized.

Copy link
Member Author

martijnvg commented Oct 10, 2013

@karussell The main reason why we moved to hppc is because Hppc has a cleaner licence. Hppc is Apache licensed and Trove is LGPL licensed. Also we noticed that hppc performs a bit faster than trove does, so that was a second but less minor reason to move to Hppc. I actually didn't know about this stability issue with the G1 garbage collector, but that seems like a positive side effect.

@karussell

This comment has been minimized.

Copy link
Contributor

karussell commented Oct 10, 2013

Thanks for the explanation! But JTS is still lgpl right?

@martijnvg

This comment has been minimized.

Copy link
Member Author

martijnvg commented Oct 10, 2013

Yes, but that is an optional library.

martijnvg referenced this issue Dec 10, 2013

Cut the following classes over to ImmutableOpenMap:
* MetaData
* IndexMetaData
* ClusterState
* IndexTemplateMetaData
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.