Skip to content

Conversation

@lcawl
Copy link
Contributor

@lcawl lcawl commented Mar 19, 2020

Related to #53737 and #53643

This PR adds missing details for the cat transforms API.

Preview: http://elasticsearch_53776.docs-preview.app.elstc.co/guide/en/elasticsearch/reference/master/cat-transforms.html

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

Pinging @elastic/es-docs (>docs)

@elasticmachine
Copy link
Collaborator

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

include::{docdir}/rest-api/common-parms.asciidoc[tag=dest-index]

`document_total`, `dt`:::
The total number of documents.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

document_total

I don't see an equivalent stat in the "get transform stats" API. Is it based on documents_processed or documents_indexed?

Choose a reason for hiding this comment

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

according to the PR it is documents_processed

include::{docdir}/rest-api/common-parms.asciidoc[tag=exponential-avg-documents-indexed]

`invocation_total`, `itotal`:::
The total number of invocations.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

invocation_total

Is this related to continuous transforms? I couldn't see a matching item in the "get transform stats" API.

Choose a reason for hiding this comment

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

this is called trigger_count in stats

include::{docdir}/rest-api/common-parms.asciidoc[tag=exponential-avg-documents-processed]

`reason`, `r`:::
The reason for the current state.
Copy link
Contributor Author

@lcawl lcawl Mar 20, 2020

Choose a reason for hiding this comment

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

reason

Does this information appear only when the state="failed"? Is it returned in those circumstances from the "get transform stats" API too?

Choose a reason for hiding this comment

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

in the json output of get transform stats its only returned if set, which is only the case of a failure.

On the tabular output of _cat it can't be omitted, so it's in the table but usually empty

@hendrikmuhs
Copy link

I think you made some fair points that the naming is inconsistent for the REST layer, the HLRC layer and the _cat layer. I think we should fix this.

@lcawl lcawl marked this pull request as ready for review March 24, 2020 00:24
@bpintea bpintea added v7.8.0 and removed v7.7.0 labels Mar 25, 2020
@lcawl lcawl merged commit 6fceef7 into elastic:master Mar 25, 2020
@lcawl lcawl deleted the cat-transforms-more branch March 25, 2020 19:42
lcawl added a commit to lcawl/elasticsearch that referenced this pull request Mar 25, 2020
lcawl added a commit to lcawl/elasticsearch that referenced this pull request Mar 25, 2020
@jakelandis jakelandis removed the v8.0.0 label Jul 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants