Skip to content

Conversation

@ywangd
Copy link
Member

@ywangd ywangd commented Oct 1, 2025

So that we can pass a pared down implementation instead of the full ClusterInfo which can be expensive sometimes.

Relates: ES-12723

So that we can pass a pared down implementation instead of the full
ClusterInfo which can be expensive sometimes.

Relates: ES-12723
@ywangd ywangd requested a review from nicktindall October 1, 2025 03:32
@ywangd ywangd added :Distributed Coordination/Allocation All issues relating to the decision making around placing a shard (both master logic & on the nodes) >refactoring v9.2.0 labels Oct 1, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Distributed Coordination Meta label for Distributed Coordination team label Oct 1, 2025
@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Contributor

@nicktindall nicktindall left a comment

Choose a reason for hiding this comment

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

Nice

@ywangd ywangd added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Oct 1, 2025
@elasticsearchmachine elasticsearchmachine merged commit 6cff547 into elastic:main Oct 1, 2025
34 checks passed
@ywangd ywangd deleted the shard-size-provider-interface branch October 1, 2025 05:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Distributed Coordination/Allocation All issues relating to the decision making around placing a shard (both master logic & on the nodes) >refactoring Team:Distributed Coordination Meta label for Distributed Coordination team v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants