Skip to content

Conversation

@joshua-adams-1
Copy link
Contributor

Removes conditional logic from DesiredBalanceResponse and all classes used in the response (DesiredBalanceStats, ClusterBalanceStats and ClusterInfo) that altered serialization based on transport
versions <= V_8_18_0 since these are outdated.

Jira: ES-10337

Removes conditional logic from `DesiredBalanceResponse` and
all classes used in the response (`DesiredBalanceStats`,
`ClusterBalanceStats` and `ClusterInfo`) that altered
serialization based on transport
versions <= `V_8_18_0` since these are outdated.

Jira: ES-10337
@joshua-adams-1 joshua-adams-1 self-assigned this Jul 23, 2025
@joshua-adams-1 joshua-adams-1 added >non-issue :Distributed Coordination/Distributed A catch all label for anything in the Distributed Coordination area. Please avoid if you can. labels Jul 23, 2025
@joshua-adams-1 joshua-adams-1 requested a review from ywangd July 23, 2025 13:40
@joshua-adams-1 joshua-adams-1 marked this pull request as ready for review July 23, 2025 13:40
@elasticsearchmachine elasticsearchmachine added the Team:Distributed Coordination Meta label for Distributed Coordination team label Jul 23, 2025
@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Member

@ywangd ywangd left a comment

Choose a reason for hiding this comment

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

LGTM

@joshua-adams-1 joshua-adams-1 merged commit 1e120b5 into elastic:main Jul 24, 2025
33 checks passed
@joshua-adams-1 joshua-adams-1 deleted the desired-balance-response-transport-version branch July 24, 2025 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed Coordination/Distributed A catch all label for anything in the Distributed Coordination area. Please avoid if you can. >non-issue 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