Skip to content

Conversation

masseyke
Copy link
Member

@masseyke masseyke commented Aug 13, 2025

This adds the new merge_type query parameter to the specification for the simulate ingest API. The parameter was added in elastic/elasticsearch#132210.

@masseyke masseyke requested a review from a team as a code owner August 13, 2025 21:18
@masseyke masseyke added the skip-backport This pull request should not be backported label Aug 13, 2025
@masseyke masseyke changed the title Adding specification for the new data stream mappings API Adding the merge_type param to the ingest simulate API Aug 13, 2025
Copy link
Contributor

Following you can find the validation changes against the target branch for the APIs.

No changes detected.

You can validate these APIs yourself by using the make validate target.

},
"merge_type": {
"type": "string",
"description": "The way that mapping_additions ought to be merged into existing mappings"
Copy link
Contributor

Choose a reason for hiding this comment

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

Just a suggestion, but thought the description from your original PR was easier for me to understand

allows a user to specify exactly how they would like the mappings merged. The options are index or template. By default, they will be merged using the index strategy to maintain backwards compatibility.

@masseyke masseyke merged commit 9317319 into main Aug 14, 2025
8 checks passed
@masseyke masseyke deleted the add-simulate-ingest-merge-type branch August 14, 2025 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

compiler skip-backport This pull request should not be backported specification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants