Skip to content

[7.x] Handle merging dotted object names when merging V2 template mappings (#55982)#56041

Merged
dakrone merged 2 commits intoelastic:7.xfrom
dakrone:backport/7.x/pr-55982
Apr 30, 2020
Merged

[7.x] Handle merging dotted object names when merging V2 template mappings (#55982)#56041
dakrone merged 2 commits intoelastic:7.xfrom
dakrone:backport/7.x/pr-55982

Conversation

@dakrone
Copy link
Copy Markdown
Member

@dakrone dakrone commented Apr 30, 2020

Backports the following commits to 7.x:

…lastic#55982)

When merging component template, index template, and request mappings, we now treat any declaration
of a top-level field the same as replacing all sub-objects. For example, assuming two component
templates with mappings and template B taking precedence:

```
A: {foo: {...}}
B: {foo.bar: {...}}
Result: {foo.bar: {...}}

A: {foo.bar: {...}}
B: {foo: {...}}
Result: {foo: {...}}

A: {foo.bar: {...}}
B: {foo.baz: {...}}
Result: {foo.baz: {...}}
```

Relates to elastic#53101
@dakrone dakrone added backport :Data Management/Indices APIs DO NOT USE. Use ":Distributed/Indices APIs" or ":StorageEngine/Templates" instead. v7.8.0 labels Apr 30, 2020
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-core-features (:Core/Features/Indices APIs)

@elasticmachine elasticmachine added the Team:Data Management (obsolete) DO NOT USE. This team no longer exists. label Apr 30, 2020
@dakrone dakrone merged commit 3dada1e into elastic:7.x Apr 30, 2020
@dakrone dakrone deleted the backport/7.x/pr-55982 branch April 30, 2020 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport :Data Management/Indices APIs DO NOT USE. Use ":Distributed/Indices APIs" or ":StorageEngine/Templates" instead. Team:Data Management (obsolete) DO NOT USE. This team no longer exists. v7.8.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants