Skip to content

Document PUT as the prefered HTTP method for index API #45622

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

Conversation

Mpdreamz
Copy link
Member

Follow up from #42346. Since the methods array is in order of
preference when calling the index API with an {id} we prefer to use
the PUT http method.

cc @elastic/es-clients

Follow up from elastic#42346. Since the `methods` array is in order of
preference when calling the index API with an `{id}` we prefer to use
the `PUT` http method.
@Mpdreamz Mpdreamz added the :Core/Infra/REST API REST infrastructure and utilities label Aug 15, 2019
@Mpdreamz Mpdreamz requested a review from javanna August 15, 2019 16:27
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

Copy link
Member

@javanna javanna left a comment

Choose a reason for hiding this comment

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

LGTM

@javanna
Copy link
Member

javanna commented Aug 15, 2019

run elasticsearch-ci/1

@colings86 colings86 added v7.5.0 and removed v7.4.0 labels Aug 30, 2019
@javanna javanna merged commit 716b4e1 into elastic:master Sep 19, 2019
javanna pushed a commit that referenced this pull request Sep 19, 2019
Follow up from #42346. Since the `methods` array is in order of
preference when calling the index API with an `{id}` we prefer to use
the `PUT` http method.
@jimczi jimczi added the >docs General docs changes label Nov 12, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (>docs)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/REST API REST infrastructure and utilities >docs General docs changes v7.5.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants