-
Notifications
You must be signed in to change notification settings - Fork 25k
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
[DOCS] Adds cat datafeeds API #52738
Conversation
Pinging @elastic/es-docs (>docs) |
Pinging @elastic/ml-core (:ml) |
@elasticmachine update branch |
`search.bucket_avg`:: | ||
include::{docdir}/ml/ml-shared.asciidoc[tag=search-bucket-avg] | ||
+ | ||
To retrieve this information, specify the `sba` or `bucketTimeMin` column in the |
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 is wrong. The API is also wrong as well. I will correct in a PR. Then we can correct it here :)
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 should be searchBucketAvg
. Could you change it here and in https://github.com/elastic/elasticsearch/blob/master/x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/rest/cat/RestCatDatafeedsAction.java#L95
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.
Good catch, thanks @benwtrent ! I've made that change
Related to #51411
This PR adds documentation for the cat datafeeds API.
Preview: http://elasticsearch_52738.docs-preview.app.elstc.co/guide/en/elasticsearch/reference/master/cat-datafeeds.html