Skip to content

[DOCS] Add processing details to get transforms stats API #54368

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

Merged
merged 4 commits into from
Apr 1, 2020

Conversation

lcawl
Copy link
Contributor

@lcawl lcawl commented Mar 27, 2020

Related to #53770

This PR adds the processing_time_in_ms and processing_total statistics to the "get transform stats" API reference (https://www.elastic.co/guide/en/elasticsearch/reference/master/get-transform-stats.html).

Preview: http://elasticsearch_54368.docs-preview.app.elstc.co/guide/en/elasticsearch/reference/master/get-transform-stats.html

@lcawl lcawl added >docs General docs changes v8.0.0 :ml/Transform Transform v7.7.0 v7.8.0 labels Mar 27, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core (:ml/Transform)

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (>docs)

@lcawl lcawl requested a review from hendrikmuhs March 30, 2020 13:52
@lcawl lcawl marked this pull request as ready for review March 30, 2020 13:52
@lcawl lcawl force-pushed the transform-stats branch from e1ab364 to 484cc1f Compare March 30, 2020 15:08
Copy link

@hendrikmuhs hendrikmuhs left a comment

Choose a reason for hiding this comment

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

LGTM, a reference needs an update

@@ -262,7 +270,7 @@ The following example gets usage information for the `ecommerce_transform`

[source,console]
--------------------------------------------------
GET _transform/ecommerce_transform/_stats
GET _transform/ecommerce-customer-transform/_stats

Choose a reason for hiding this comment

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

look like line 268 has to be changed, too:

The following example gets usage information for the ecommerce_transform

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks! I've removed that qualifier

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