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

[DOCS] Document _cat/tasks API #47321

Merged
merged 1 commit into from
Oct 1, 2019
Merged

[DOCS] Document _cat/tasks API #47321

merged 1 commit into from
Oct 1, 2019

Conversation

jrodewig
Copy link
Contributor

Changes

  • Adds separate docs page for the _cat/tasks API, similar to other cat APIs
  • Documents several query parameters for the cat and JSON tasks API

Closes #19756

@jrodewig jrodewig added >docs General docs changes :Data Management/CAT APIs Text APIs behind /_cat :Distributed Coordination/Task Management Issues for anything around the Tasks API - both persistent and node level. v8.0.0 v7.5.0 v7.4.1 v7.3.3 labels Sep 30, 2019
@jrodewig jrodewig requested a review from imotov September 30, 2019 20:06
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs

Copy link
Contributor

@imotov imotov left a comment

Choose a reason for hiding this comment

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

LGTM, left one minor comment.

<titleabbrev>cat task management</titleabbrev>
++++

beta::[The cat task management API is new and should still be considered a beta feature. The API may change in ways that are not backwards compatible.]
Copy link
Contributor

Choose a reason for hiding this comment

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

I am not totally sure why task management is still beta. Perhaps, it is time to start a discussion about removal of this label. But since the non-cat API is marked beta, it is appropriate to use this label here as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've opened #47378 to discuss this more.

[[cat-tasks-api-path-params]]
==== {api-path-parms-title}

include::{docdir}/rest-api/common-parms.asciidoc[tag=task-id]
Copy link
Contributor

Choose a reason for hiding this comment

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

Looking at the code, it should also support detailed, wait_for_completion and timeout flags. They are not as useful in the cat context but we can add them for completeness.

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! These are included under the query parameters section.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Data Management/CAT APIs Text APIs behind /_cat :Distributed Coordination/Task Management Issues for anything around the Tasks API - both persistent and node level. >docs General docs changes v7.3.3 v7.4.1 v7.5.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DOCS] Document _cat/tasks API
4 participants