Skip to content

Stop automatically nesting mappings in index creation requests (backport to 6.x). #37132

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

Closed
wants to merge 2 commits into from

Conversation

jtibshirani
Copy link
Contributor

@jtibshirani jtibshirani commented Jan 4, 2019

Backport of #36924. There was an additional serialization change required that was not present in the original PR.

@jtibshirani jtibshirani added >bug :Search Foundations/Mapping Index mappings, including merging and defining field types backport labels Jan 4, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search

@jtibshirani jtibshirani force-pushed the create-index-mappings branch from fbfd8ad to 59ac655 Compare January 4, 2019 02:19
…tic#36924)

Now that we unwrap mappings in DocumentMapperParser#extractMappings, it is not
necessary for the mapping definition to always be nested under the type. This
leniency around the mapping format was added in 2341825.
@jtibshirani jtibshirani force-pushed the create-index-mappings branch from 59ac655 to 8baf144 Compare January 4, 2019 02:47
@jtibshirani jtibshirani requested a review from jpountz January 4, 2019 21:23
@jtibshirani jtibshirani force-pushed the create-index-mappings branch from fdca865 to 8bd2d8d Compare January 4, 2019 21:26
@jtibshirani
Copy link
Contributor Author

Closing this PR for now, since we ran into issues with the original change on master.

@jtibshirani jtibshirani closed this Jan 8, 2019
@jtibshirani jtibshirani deleted the create-index-mappings branch January 8, 2019 01:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport >bug :Search Foundations/Mapping Index mappings, including merging and defining field types
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants