Skip to content

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

Merged
merged 1 commit into from
Nov 10, 2021

Conversation

dnhatn
Copy link
Member

@dnhatn dnhatn commented Nov 10, 2021

This setting has been removed in Lucene 9.

@dnhatn dnhatn added >deprecation :Distributed Indexing/Engine Anything around managing Lucene and the Translog in an open shard. v8.0.0 auto-backport Automatically create backport pull requests when merged v8.1.0 labels Nov 10, 2021
@dnhatn dnhatn requested a review from romseygeek November 10, 2021 03:03
@elasticmachine elasticmachine added the Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. label Nov 10, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (Team:Distributed)

@dnhatn dnhatn mentioned this pull request Nov 10, 2021
16 tasks
Copy link
Contributor

@romseygeek romseygeek left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @dnhatn

@dnhatn
Copy link
Member Author

dnhatn commented Nov 10, 2021

Thanks Alan!

@dnhatn dnhatn merged commit fa8f1b4 into elastic:master Nov 10, 2021
@dnhatn dnhatn deleted the deprecate_max_merge_at_once_explicit branch November 10, 2021 13:20
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.0

dnhatn added a commit to dnhatn/elasticsearch that referenced this pull request Nov 10, 2021
This setting has been removed in Lucene 9.
dnhatn added a commit that referenced this pull request Nov 10, 2021
This setting has been removed in Lucene 9.
@dakrone
Copy link
Member

dakrone commented Nov 10, 2021

@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.

@jpountz
Copy link
Contributor

jpountz commented Nov 23, 2021

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?

@jpountz
Copy link
Contributor

jpountz commented Nov 24, 2021

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 warning level instead of critical, as a way to convey information about the fact that the behavior will change in future versions. So I guess we need to add this setting there then.

arteam added a commit to arteam/elasticsearch that referenced this pull request Mar 13, 2025
…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
arteam added a commit that referenced this pull request Mar 14, 2025
…ed in v10 (#124807)

Introduce `Property.IndexSettingDeprecatedInV9AndRemovedInV10` property setting to mark
index settings that accepted on 9.x indices, but not on 10.0.

See #80574, #90264
See ES-11224
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Automatically create backport pull requests when merged >deprecation :Distributed Indexing/Engine Anything around managing Lucene and the Translog in an open shard. Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. v8.0.0-rc2 v8.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants