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

VALIDATOR-390: remove FastHashMap and replace with ConcurrentHashMap #133

Closed
wants to merge 3 commits into from
Closed

VALIDATOR-390: remove FastHashMap and replace with ConcurrentHashMap #133

wants to merge 3 commits into from

Conversation

LoreScianatico
Copy link

No description provided.

@garydgregory
Copy link
Member

@LoreScianatico
-1: Don't break binary compatibility.

@LoreScianatico
Copy link
Author

@LoreScianatico -1: Don't break binary compatibility.

How do I check binary compatibility?

@LoreScianatico
Copy link
Author

How can we get rid of FastHashMap? It's not even present in newest commons-collections

@garydgregory
Copy link
Member

@LoreScianatico -1: Don't break binary compatibility.

How do I check binary compatibility?

You have to know the Java language, or let this build check for you, in this case, a break of BC will cause the build to fail.
Please see also https://garygregory.wordpress.com/2020/06/14/how-we-handle-binary-compatibility-at-apache-commons/

@garydgregory
Copy link
Member

How can we get rid of FastHashMap? It's not even present in newest commons-collections

You can't "get rid" of it within a minor release.

A major release can choose to break BC, which then comes along with a change in package name and Maven coordinates.

@LoreScianatico
Copy link
Author

Ok, this is clear (and useful). Is there any plan for a major release?

@garydgregory
Copy link
Member

Ok, this is clear (and useful). Is there any plan for a major release?

Not yet, there are other Apache Commons components that need attention first.

@LoreScianatico
Copy link
Author

Ok, this is clear (and useful). Is there any plan for a major release?

Not yet, there are other Apache Commons components that need attention first.

All right... I guess beanutils is one of those components. Is there any roadmap where I can pick stuff to contribute?

@garydgregory
Copy link
Member

garydgregory commented May 26, 2023

Ok, this is clear (and useful). Is there any plan for a major release?

Not yet, there are other Apache Commons components that need attention first.

All right... I guess beanutils is one of those components. Is there any roadmap where I can pick stuff to contribute?

There is no Commons-wide roadmap, it's what you can see on the developer's mailing list. You can also look at Jira.

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