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

[Rest API Compatibility] Typed endpoints for Put and Get Mapping and get field mappings #71721

Merged
merged 52 commits into from
May 10, 2021

Conversation

pgomulka
Copy link
Contributor

@pgomulka pgomulka commented Apr 15, 2021

Implements a V7 compatible typed endpoints for REST put and get mapping endpoints. Also for Get Field Mappings endpoints.
retrofits the REST layer change removed in #41676

relates main meta issue #51816
relates types removal issue #54160

@pgomulka pgomulka marked this pull request as ready for review April 19, 2021 18:42
@pgomulka pgomulka added :Core/Infra/REST API REST infrastructure and utilities v8.0.0 labels Apr 19, 2021
@elasticmachine elasticmachine added the Team:Core/Infra Meta label for core/infra team label Apr 19, 2021
@elasticmachine
Copy link
Collaborator

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

@pgomulka pgomulka changed the title draft [Rest API Compatibility] Typed endpoints for Put and Get Mapping and get field mappings [Rest API Compatibility] Typed endpoints for Put and Get Mapping and get field mappings Apr 19, 2021
@pgomulka pgomulka removed the WIP label Apr 19, 2021
@pgomulka
Copy link
Contributor Author

pgomulka commented Apr 20, 2021

once again, this PR includes a testing changes which will be removed later. I will merge them on a separate PR earlier. Feel free to ignore them.

Copy link
Contributor

@romseygeek romseygeek left a comment

Choose a reason for hiding this comment

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

I can't comment on the test rewrite code but LGTM from the mappings point of view.

import org.gradle.api.tasks.Internal;

/**
* A transformation to replace the value of a match. For example, change from "match":{"_type": "foo"} to "match":{"_type": "bar"}
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the docstring here is not accurate.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, I submited a separate PR for testing part here #72156

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 >enhancement Team:Core/Infra Meta label for core/infra team v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants