[7.8] ITV2: disallow duplicate dynamic templates (#56291)#56614
Merged
andreidan merged 3 commits intoelastic:7.8from May 14, 2020
Merged
[7.8] ITV2: disallow duplicate dynamic templates (#56291)#56614andreidan merged 3 commits intoelastic:7.8from
andreidan merged 3 commits intoelastic:7.8from
Conversation
Dynamic templates can contain multiple templates with the same name. This commit changes the way V2 index templates mappings are resolved to add deduplication of dynamic templates when resolving the mappings, by having the last dynamic templates override the ones defined with the same name earlier in the `dynamic_templates` array. This also filters duplicate dynamic templates when merging different component templates that specify the same dynamic template and when merging the mappings specified in the request with the ones in the index template. (cherry picked from commit eb4a557) Signed-off-by: Andrei Dan <andrei.dan@elastic.co>
Collaborator
|
Pinging @elastic/es-core-features (:Core/Features/Indices APIs) |
Contributor
Author
|
@elasticmachine update branch |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Dynamic templates can contain multiple templates with the same name. This
commit changes the way V2 index templates mappings are resolved to add
deduplication of dynamic templates when resolving the mappings, by having
the last dynamic templates override the ones defined with the same name
earlier in the
dynamic_templatesarray.This also filters duplicate dynamic templates when merging different component
templates that specify the same dynamic template and when merging the
mappings specified in the request with the ones in the index template.
(cherry picked from commit eb4a557)
Signed-off-by: Andrei Dan andrei.dan@elastic.co
Backport of #56291