Skip to content
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

Disallow _field_names enabled setting #46681

Merged
merged 5 commits into from
Sep 23, 2019

Conversation

cbuescher
Copy link
Member

After deprecating the setting starting with 7.5 in #42854, this change disallows
setting enabled for the _field_names field on new indices. The setting
continues to work for indices created in 7.x where we still emit a deprecation
warning.

After deprecating the setting starting with 7.5 in elastic#42854, this change disallows
setting enabled for the `_field_names` field on new indices. The setting
continues to work for indices created in 7.x where we still emit a deprecation
warning.
@cbuescher cbuescher added >breaking :Search Foundations/Mapping Index mappings, including merging and defining field types >refactoring v8.0.0 labels Sep 12, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search

Copy link
Contributor

@jtibshirani jtibshirani left a comment

Choose a reason for hiding this comment

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

Looks good to me -- just made a couple notes about typos that would be good to fix before merging.

@cbuescher
Copy link
Member Author

@elasticmachine run elasticsearch-ci/2

@cbuescher
Copy link
Member Author

@elasticmachine run elasticsearch-ci/2

@cbuescher cbuescher merged commit 2351aa3 into elastic:master Sep 23, 2019
cbuescher pushed a commit to cbuescher/elasticsearch that referenced this pull request Sep 23, 2019
This change adds a check to the migration tool that warns about the deprecated
"enabled" setting for the "_field_names" field on 7.x indices and issues a
critical error for templates containing this setting, since it has been removed
with 8.0.

Relates to elastic#42854, elastic#46681
cbuescher pushed a commit that referenced this pull request Sep 25, 2019
This change adds a check to the migration tool that warns about the deprecated
`enabled` setting for the `_field_names` field on 7.x indices and issues a
warning for templates containing this setting, which has been removed
with 8.0.

Relates to #42854, #46681
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>breaking >refactoring :Search Foundations/Mapping Index mappings, including merging and defining field types v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants