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

Expose external refreshes through the stats API (#38643) #40395

Merged
merged 1 commit into from
Mar 25, 2019

Conversation

dnhatn
Copy link
Member

@dnhatn dnhatn commented Mar 23, 2019

Right now, the stats API only provides refresh metrics regarding
internal refreshes. This isn't very useful and somewhat misleading for
cluster administrators since the internal refreshes are not indicative
of documents being available for search.

In this PR I added a new metric for collecting external refreshes as
they occur and exposing them through the stats API. Now, calling an
endpoint for stats will yield external refresh metrics as well.

Relates #36712
Backport of #38643

Right now, the stats API only provides refresh metrics regarding
internal refreshes. This isn't very useful and somewhat misleading for
cluster administrators since the internal refreshes are not indicative
of documents being available for search.

In this PR I added a new metric for collecting external refreshes as
they occur and exposing them through the stats API. Now, calling an
endpoint for stats will yield external refresh metrics as well.

Relates elastic#36712
@dnhatn
Copy link
Member Author

dnhatn commented Mar 24, 2019

run elasticsearch-ci/2

@dnhatn dnhatn merged commit b9f96a8 into elastic:7.x Mar 25, 2019
@dnhatn dnhatn deleted the external-refresh-7 branch March 25, 2019 02:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant