-
Notifications
You must be signed in to change notification settings - Fork 25.3k
Deprecate setting max_merge_at_once_explicit
#80574
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
Deprecate setting max_merge_at_once_explicit
#80574
Conversation
Pinging @elastic/es-distributed (Team:Distributed) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks @dnhatn
Thanks Alan! |
💚 Backport successful
|
This setting has been removed in Lucene 9.
This setting has been removed in Lucene 9.
@dnhatn would you be able to add this setting to the deprecation API in a follow-up PR? That way users start getting notified about it sooner rather than later, and it lowers the chance that we forget about adding it at a later time. |
I wonder if we actually should add it to the deprecation API. It doesn't hurt to have this setting set on old indices, it will just be ignored with this PR, and then archived once we effectively remove the setting. So not having it in the deprecation API could be a nicer user experience: less noise in the deprecation API and Elasticsearch takes care of the migration for existing indices? |
Reading more about the migration API it looks like it's still expected to cover changes that users do not need to take action on, using the |
…ed in v10 Introduce `Property.IndexSettingDeprecatedInV9AndRemovedInV10` property setting to mark index settings that accepted on 9.x indices, but not on 10.0. See elastic#80574, elastic#90264 See ES-11224
This setting has been removed in Lucene 9.