Skip to content

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