Skip to content

Remove ImmutableOpenIntMap from ClusterStateHealthTests#86306

Merged
rjernst merged 1 commit intoelastic:masterfrom
rjernst:hppc/cluster_state_health_tests
May 2, 2022
Merged

Remove ImmutableOpenIntMap from ClusterStateHealthTests#86306
rjernst merged 1 commit intoelastic:masterfrom
rjernst:hppc/cluster_state_health_tests

Conversation

@rjernst
Copy link
Copy Markdown
Member

@rjernst rjernst commented Apr 29, 2022

These tests construct a map of shard to allocation ids. But they only do
so get the pairs to put into the index metadata builder. This commit
converts it to use HashMap.

relates #86239

These tests construct a map of shard to allocation ids. But they only do
so get the pairs to put into the index metadata builder. This commit
converts it to use HashMap.

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

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

Copy link
Copy Markdown
Contributor

@williamrandolph williamrandolph 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 fbe1c59 into elastic:master May 2, 2022
@rjernst rjernst deleted the hppc/cluster_state_health_tests branch May 2, 2022 13:52
@rjernst rjernst mentioned this pull request May 2, 2022
35 tasks
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