Skip to content

Optimize unchanged metadata handling in PersistedClusterStateService#90164

Merged
original-brownbear merged 1 commit intoelastic:mainfrom
original-brownbear:optimize-metadata-unchanged
Sep 20, 2022
Merged

Optimize unchanged metadata handling in PersistedClusterStateService#90164
original-brownbear merged 1 commit intoelastic:mainfrom
original-brownbear:optimize-metadata-unchanged

Conversation

@original-brownbear
Copy link
Copy Markdown
Contributor

With #90101 both follower and leader nodes will get instance equality on the metadata on non-metadata changes here reliably. We can leverage that and skip the expensive loop over all indices in this method on instance equality.

relates #77466

With #90101 both follower and leader nodes will get instance equality
on the metadata on non-metadata changes here reliably.
We can leverage that and skip the expensive loop over all indices
in this method on instance equality.
@original-brownbear original-brownbear added >non-issue :Distributed/Cluster Coordination Cluster formation and cluster state publication, including cluster membership and fault detection. v8.5.0 labels Sep 20, 2022
@elasticsearchmachine elasticsearchmachine added the Team:Distributed Meta label for distributed team. label Sep 20, 2022
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

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

Copy link
Copy Markdown
Member

@DaveCTurner DaveCTurner left a comment

Choose a reason for hiding this comment

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

LGTM

@original-brownbear
Copy link
Copy Markdown
Contributor Author

Thanks David!

@original-brownbear original-brownbear merged commit c462e50 into elastic:main Sep 20, 2022
@original-brownbear original-brownbear deleted the optimize-metadata-unchanged branch September 20, 2022 12:09
@original-brownbear original-brownbear restored the optimize-metadata-unchanged branch November 30, 2024 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Cluster Coordination Cluster formation and cluster state publication, including cluster membership and fault detection. >non-issue Team:Distributed Meta label for distributed team. v8.5.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants