Skip to content

Deprecate local param in get_mapping.json #57265

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

russcam
Copy link
Contributor

@russcam russcam commented May 28, 2020

Relates: #55014

This commit deprecates the local param in get_mapping.json.
This parameter is a no-op and field mappings are always retrieved locally.

Relates: elastic#55014

This commit deprecates the local param in get_mapping.json.
This parameter is a no-op and field mappings are always retrieved locally.
@russcam russcam requested review from martijnvg and ywangd May 28, 2020 00:03
Copy link
Member

@ywangd ywangd left a comment

Choose a reason for hiding this comment

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

LGTM. Didn't know we could have deprecation in the API spec. TIL.

@dliappis dliappis added the :Core/Infra/REST API REST infrastructure and utilities label May 28, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (:Core/Infra/REST API)

@elasticmachine elasticmachine added the Team:Core/Infra Meta label for core/infra team label May 28, 2020
@dliappis dliappis added the Team:Clients Meta label for clients team label May 28, 2020
Copy link
Member

@martijnvg martijnvg left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@russcam russcam merged commit 0b041cc into elastic:master May 29, 2020
@russcam russcam deleted the rest-specs/deprecate-get-mapping-local branch May 29, 2020 02:24
russcam added a commit that referenced this pull request May 29, 2020
Relates: #55014

This commit deprecates the local param in get_mapping.json.
This parameter is a no-op and field mappings are always retrieved locally.

(cherry picked from commit 0b041cc)
@russcam
Copy link
Contributor Author

russcam commented May 29, 2020

ported to 7.x.
Any objections to porting this to 7.8?

nielsbauman added a commit to nielsbauman/elasticsearch that referenced this pull request Feb 19, 2025
The `local` param for the `GetFieldMapping` API was deprecated in elastic#55014
and I think elastic#57265 aimed to propogate that deprecation to the REST API
spec, but it changed `get_mapping.json` instead of
`get_field_mapping.json`. elastic#55100 removed the `local` param for the
_field_ mapping API so we can safely remove the field from the spec and
remove the YAML test.
nielsbauman added a commit that referenced this pull request Feb 19, 2025
The `local` param for the `GetFieldMapping` API was deprecated in #55014
and I think #57265 aimed to propogate that deprecation to the REST API
spec, but it changed `get_mapping.json` instead of
`get_field_mapping.json`. #55100 removed the `local` param for the
_field_ mapping API so we can safely remove the field from the spec and
remove the YAML test.
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 Team:Clients Meta label for clients team Team:Core/Infra Meta label for core/infra team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants