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

Convert cluster state "customs" to Map #87265

Merged
merged 2 commits into from
Jun 1, 2022

Conversation

rjernst
Copy link
Member

@rjernst rjernst commented May 31, 2022

The pluggable portions of in memory cluster state are held within an
ImmutableOpenMap. This commit converts that to a Map. Note that the diff
format is the same for immutableopenmap and jdk map, so no bwc logic is
necessary.

relates #86239

The pluggable portions of in memory cluster state are held within an
ImmutableOpenMap. This commit converts that to a Map. Note that the diff
format is the same for immutableopenmap and jdk map, so no bwc logic is
necessary.

relates elastic#86239
@rjernst rjernst added :Distributed Indexing/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. >refactoring v8.4.0 labels May 31, 2022
@elasticmachine elasticmachine added the Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. label May 31, 2022
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (Team:Distributed)

@rjernst rjernst merged commit 767fd68 into elastic:master Jun 1, 2022
@rjernst rjernst deleted the hppc/cluster_state_customs branch June 1, 2022 12:45
@rjernst rjernst mentioned this pull request Jun 1, 2022
35 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed Indexing/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. >refactoring Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. v8.4.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants