Skip to content
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

Rename 'indices stats API' to 'index stats API' #1422

Merged
merged 1 commit into from
Jan 31, 2022

Conversation

pquentin
Copy link
Member

It was renamed in 2019 and is documented as "index stats" since 7.3: https://www.elastic.co/guide/en/elasticsearch/reference/7.3/indices-stats.html. (The URL did not change, but the title did.)

I'm not changing the node-stats-include-indices to avoid having a lengthy migration. Should I have done that?

@pquentin pquentin added the :Docs Changes to the documentation label Jan 28, 2022
@pquentin pquentin added this to the 2.4.0 milestone Jan 28, 2022
@pquentin pquentin requested review from dliappis and inqueue January 28, 2022 12:40
@pquentin pquentin self-assigned this Jan 28, 2022
Copy link
Contributor

@dliappis dliappis left a comment

Choose a reason for hiding this comment

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

LGTM thank you!

@pquentin pquentin merged commit d5d16f1 into elastic:master Jan 31, 2022
@pquentin pquentin deleted the index-stats-api branch January 31, 2022 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Docs Changes to the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants