-
Notifications
You must be signed in to change notification settings - Fork 102
ES-10037 Add new write load metrics to IndexingStats
#4195
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
ES-10037 Add new write load metrics to IndexingStats
#4195
Conversation
This adds the fields added in elastic/elasticsearch#125521 and elastic/elasticsearch#124652 to the spec.
IndexingStats
IndexingStats
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, thanks Pete!
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.
Actually, I believe you need to make generate
the output changes also
@pquentin would it be possible to have something like |
I believe that there's a bot which periodically comes along and commits any needed output changes. But it's not part of the same PR. (I've done |
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 still :)
@dakrone As @PeteGillinElastic mentioned, it's OK not to run those commands, as CI will notice if there's an issue and commit the output/ files. A pre-commit hook is an interesting idea, are you suggesting that I add that script to the repository and explain in CONTRIBUTING.md how to install it? |
Ah okay, I hadn't realized that CI would do the commit for it. As for the pre-commit, what you described is what I was suggesting, but I don't think it's necessary unless we have to have the |
This adds the fields added in
elastic/elasticsearch#125521 and elastic/elasticsearch#124652 to the spec.