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

metric on watcher stats is a list not an enum #39114

Merged
merged 1 commit into from
Mar 4, 2019

Conversation

Mpdreamz
Copy link
Member

enum is a single option from a known list of options
list is an array of unknown values
flags are multiple options from a list of known options.

We don't support the flags type but a list with options acts as one. This is already the case for other API's taking metric such as node.stats.json.

watcher.stats behaves the same as other API's as metrics and as such accepts the following GET _xpack/watcher/stats/queued_watches,current_watches

cc @elastic/es-clients

`enum` is a single option from a known list of `options`
`list` is an array of unknown values
`flags` are multiple options from a list of known `options`.

We don't support the `flags` type but a `list` with `options` acts as one. This is already the case for other API's taking metric such as `node.stats.json`. 

watcher.stats behaves the same as other API's as `metrics` and as such accepts the following `GET _xpack/watcher/stats/queued_watches,current_watches`
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features

Copy link
Contributor

@jakelandis jakelandis left a comment

Choose a reason for hiding this comment

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

LGTM, do we have any documentation that describes the differences like in your comment ?

@Mpdreamz
Copy link
Member Author

I will submit an additional PR to update the rest-spec README

Also will need some love if #39063 makes it in.

Thanks for the review @jakelandis!

@Mpdreamz Mpdreamz merged commit 4c00a02 into master Mar 4, 2019
Mpdreamz added a commit to Mpdreamz/elasticsearch that referenced this pull request Mar 4, 2019
`enum` is a single option from a known list of `options`
`list` is an array of unknown values
`flags` are multiple options from a list of known `options`.

We don't support the `flags` type but a `list` with `options` acts as one. This is already the case for other API's taking metric such as `node.stats.json`. 

watcher.stats behaves the same as other API's as `metrics` and as such accepts the following `GET _xpack/watcher/stats/queued_watches,current_watches`

(cherry picked from commit 4c00a02)
Mpdreamz added a commit to Mpdreamz/elasticsearch that referenced this pull request Mar 4, 2019
`enum` is a single option from a known list of `options`
`list` is an array of unknown values
`flags` are multiple options from a list of known `options`.

We don't support the `flags` type but a `list` with `options` acts as one. This is already the case for other API's taking metric such as `node.stats.json`. 

watcher.stats behaves the same as other API's as `metrics` and as such accepts the following `GET _xpack/watcher/stats/queued_watches,current_watches`

(cherry picked from commit 4c00a02)
Mpdreamz added a commit that referenced this pull request Mar 4, 2019
`enum` is a single option from a known list of `options`
`list` is an array of unknown values
`flags` are multiple options from a list of known `options`.

We don't support the `flags` type but a `list` with `options` acts as one. This is already the case for other API's taking metric such as `node.stats.json`. 

watcher.stats behaves the same as other API's as `metrics` and as such accepts the following `GET _xpack/watcher/stats/queued_watches,current_watches`

(cherry picked from commit 4c00a02)
Mpdreamz added a commit that referenced this pull request Mar 4, 2019
`enum` is a single option from a known list of `options`
`list` is an array of unknown values
`flags` are multiple options from a list of known `options`.

We don't support the `flags` type but a `list` with `options` acts as one. This is already the case for other API's taking metric such as `node.stats.json`. 

watcher.stats behaves the same as other API's as `metrics` and as such accepts the following `GET _xpack/watcher/stats/queued_watches,current_watches`

(cherry picked from commit 4c00a02)
@colings86 colings86 removed the :Core/Infra/REST API REST infrastructure and utilities label Jun 18, 2019
@colings86 colings86 deleted the fix/rest-spec-watcher-stats-metric branch May 27, 2020 07:40
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.

4 participants