Skip to content

Conversation

@rjernst
Copy link
Member

@rjernst rjernst commented Apr 1, 2022

The internals of ClusterStatsNodes keep a mapping from unique
jvm/architecture/etc to the count of nodes with a particular value.
These are small maps, and there is no need for them to use hppc classes.
This commit converts them to normal HashMaps.

relates #84735

The internals of ClusterStatsNodes keep a mapping from unique
jvm/architecture/etc to the count of nodes with a particular value.
These are small maps, and there is no need for them to use hppc classes.
This commit converts them to normal HashMaps.

relates elastic#84735
@rjernst rjernst added :Core/Infra/Core Core issues without another label >refactoring v8.3.0 labels Apr 1, 2022
@elasticmachine elasticmachine added the Team:Core/Infra Meta label for core/infra team label Apr 1, 2022
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@rjernst rjernst mentioned this pull request Apr 1, 2022
43 tasks
Copy link
Contributor

@grcevski grcevski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@rjernst rjernst merged commit 737eace into elastic:master Apr 4, 2022
@rjernst rjernst deleted the hppc/clusterstats_nodes branch April 4, 2022 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Core/Infra/Core Core issues without another label >refactoring Team:Core/Infra Meta label for core/infra team v8.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants