-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Index Management] Added features
parameter to ES request fetching indices
#124163
Conversation
…e ES performance
Pinging @elastic/platform-deployment-management (Team:Deployment Management) |
@elasticmachine merge upstream |
💚 Build SucceededMetrics [docs]
History
To update your PR or re-run it, just comment with: |
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.
LGTM! Tested locally and works as expected 👍
I also noticed the missing data in the flyout and then saw that we have an issue for it (#123714).
Thank you, @sebelga! |
💔 Backport failedThe pull request could not be backported due to the following error: How to fixRe-run the backport manually:
Questions ?Please refer to the Backport tool documentation |
Actual version released is 8.2.0 |
Summary
This PR adds the parameter
features
when requesting the list of all indices from Elasticsearch that has been exposed by the ES team to improve performance of the Get Index API. This option allows to specify which index properties need to be computed when getting index information:aliases
,settings
and/ormappings
. For Index Management request, we don't need to compute mappings.Note: Related to #123629 that added
filter_path
that allows to specify which index properties need to be included in the ES response but even when excluded index properties were still being calculated in ES.