Skip to content

Conversation

@gaobinlong
Copy link
Contributor

The parameter wait_for_completion 's default value should be false, but is true now.

@tvernum tvernum added the >docs General docs changes label Dec 3, 2020
@elasticmachine elasticmachine added the Team:Docs Meta label for docs team label Dec 3, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

@jrodewig
Copy link
Contributor

jrodewig commented Dec 3, 2020

Thanks for raising this @gaobinlong. However, I believe the wait_for_completion default varies based on the REST API.

To simplify things, I pushed 21c22bd to update docs containing the parameter and get rid of the reuse.

@debadair Do you mind double-checking my work here? If everything looks good, I'll merge this in.

@jrodewig
Copy link
Contributor

jrodewig commented Dec 3, 2020

@elasticmachine test this please

@jrodewig jrodewig self-requested a review December 3, 2020 15:17
@jrodewig jrodewig added the :Data Management/Indices APIs APIs to create and manage indices and templates label Dec 3, 2020
@elasticmachine elasticmachine added the Team:Data Management Meta label for data/management team label Dec 3, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (Team:Core/Features)

@jrodewig jrodewig changed the title [DOCS] Correct the default value of wait_for_completion in common par… [DOCS] Correct the default value of wait_for_completion query param Dec 3, 2020
@jrodewig jrodewig self-assigned this Dec 3, 2020
Copy link
Contributor

@debadair debadair left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for taking care of this! Reuse is great...except when it isn't.

@jrodewig jrodewig merged commit e198bb2 into elastic:master Dec 4, 2020
jrodewig added a commit that referenced this pull request Dec 4, 2020
jrodewig added a commit that referenced this pull request Dec 4, 2020
@gaobinlong gaobinlong deleted the fix_docs branch December 9, 2020 05:05
@jrodewig jrodewig removed their assignment Feb 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Data Management/Indices APIs APIs to create and manage indices and templates >docs General docs changes Team:Data Management Meta label for data/management team Team:Docs Meta label for docs team v7.10.2 v7.11.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants