Skip to content

Conversation

@jtibshirani
Copy link
Contributor

Backport of #48971, opening to run a full build.

…ic#48971)

This PR makes the following two fixes around updating flattened fields:

* Make sure that the new value for ignore_above is immediately taken into
  affect. Previously we recorded the new value but did not use it when parsing
  documents.
* Allow depth_limit to be updated dynamically. It seems plausible that a user
  might want to tweak this setting as they encounter more data.
@jtibshirani jtibshirani added >bug :Search Foundations/Mapping Index mappings, including merging and defining field types backport labels Nov 13, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search (:Search/Mapping)

@jtibshirani jtibshirani changed the title Ensure parameters are updated when merging flattened mappings. () Ensure parameters are updated when merging flattened mappings. Nov 13, 2019
@jtibshirani jtibshirani merged commit 37fa3fb into elastic:7.x Nov 13, 2019
@jtibshirani jtibshirani deleted the merge-flattened-fields branch November 13, 2019 02:50
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