Skip to content

Conversation

@thecoop
Copy link
Member

@thecoop thecoop commented Jul 5, 2023

Add index_version field to MainResponse

@thecoop thecoop added :Core/Infra/REST API REST infrastructure and utilities >refactoring labels Jul 5, 2023
@elasticsearchmachine elasticsearchmachine added v8.10.0 Team:Core/Infra Meta label for core/infra team labels Jul 5, 2023
@elasticsearchmachine
Copy link
Collaborator

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

super(in);
nodeName = in.readString();
version = Version.readVersion(in);
transportVersion = in.getTransportVersion().onOrAfter(TransportVersion.V_8_500_014) ? TransportVersion.readVersion(in) : null;
Copy link
Member Author

Choose a reason for hiding this comment

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

This should have been 19 (was already fixed in serialization)

@thecoop thecoop requested a review from rjernst July 5, 2023 08:52
Copy link
Member

@rjernst rjernst left a comment

Choose a reason for hiding this comment

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

LGTM

@thecoop thecoop merged commit 572716f into elastic:main Jul 7, 2023
@thecoop thecoop deleted the indexversion-mainresponse branch July 7, 2023 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Core/Infra/REST API REST infrastructure and utilities >refactoring Team:Core/Infra Meta label for core/infra team v8.10.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants