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
Make aggregations CacheRecycler-free. #5419
Make aggregations CacheRecycler-free. #5419
Conversation
Aggregations were still using CacheRecycler on the reduce phase. They are now using page-based recycling for both the aggregation phase and the reduce phase. Close elastic#4929
/** | ||
* Base implementation for a hash table that recycles arrays and grows in-place. | ||
*/ | ||
abstract class AbstractHashTable implements Releasable { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe rename this to AbstractReleasableHash
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wouldn't do that - it already is a
Releasable so we don't need to put it in the name
Agree, in this case it doesn't really embodies the purpose of this class... Maybe |
Oops |
As I was saying.... Maybe AbstractPagedHash? I just want to move it away as far as possible from java collections names |
@uboness just added a commit following your recommendation |
wonderful! |
LGTM! +1 |
Aggregations were still using CacheRecycler on the reduce phase. They are now
using page-based recycling for both the aggregation phase and the reduce phase.
Close #4929