-
Notifications
You must be signed in to change notification settings - Fork 25.6k
Search idle info transport version bump #96213
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
Search idle info transport version bump #96213
Conversation
Extend the Index Stats API to include information about shard idleness. To be more precise we include two pieces of info: * a boolean indicating if a shard is idle at the moment the API call takes place * a long value indicating the time in milliseconds the shard has been search idle since the last time it went into search idle state
|
Hi @salvatore-campagna, I've created a changelog YAML for you. |
|
Pinging @elastic/es-analytics-geo (Team:Analytics) |
|
Pinging @elastic/es-search (Team:Search) |
| public static final TransportVersion V_8_500_000 = registerTransportVersion(8_500_000, "dc3cbf06-3ed5-4e1b-9978-ee1d04d235bc"); | ||
| public static final TransportVersion V_8_500_001 = registerTransportVersion(8_500_001, "c943cfe5-c89d-4eae-989f-f5f4537e84e0"); | ||
| public static final TransportVersion V_8_500_002 = registerTransportVersion(8_500_002, "055dd314-ff40-4313-b4c6-9fccddfa42a8"); | ||
| public static final TransportVersion V_8_500_003 = registerTransportVersion(8_500_003, "179e81fd-c65b-4d87-b739-fa35cc3d4ee9"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder why the previous one was included but never used.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder why the previous one was included but never used.
It's used in serverless.
|
@salvatore-campagna test this please |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
This PR is the same as #95740 but also includes using the new "serverless-compatible" TransportVersion and the new version added according to our bumping rules. The previous PR was reverted by #96204 because it was breaking serverless.