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

Freezing an index should increase its index settings version #37813

Merged
merged 1 commit into from
Jan 25, 2019

Conversation

tlrx
Copy link
Member

@tlrx tlrx commented Jan 24, 2019

When an index is frozen, two index settings are updated (index.frozen and index.search.throttled) but the settings version is left unchanged and does not reflect the settings update. This pull request change the TransportFreezeIndexAction so that it also increases the settings version when an index is frozen/unfrozen.

This issue has been caught while working on the replication of closed indices (#3388) in which index metadata for a closed index are updated to frozen metadata and this specific assertion tripped.

@tlrx tlrx added >non-issue v7.0.0 :Distributed/Distributed A catch all label for anything in the Distributed Area. If you aren't sure, use this one. v6.7.0 labels Jan 24, 2019
@tlrx tlrx requested a review from ywelsch January 24, 2019 12:20
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

Copy link
Member

@dnhatn dnhatn left a comment

Choose a reason for hiding this comment

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

LGTM.

@tlrx
Copy link
Member Author

tlrx commented Jan 25, 2019

@elasticmachine run elasticsearch-ci/2

@tlrx tlrx merged commit a3baa8f into elastic:master Jan 25, 2019
@tlrx tlrx deleted the freeze-should-increase-settings-version branch January 25, 2019 10:27
@tlrx
Copy link
Member Author

tlrx commented Jan 25, 2019

Thanks @dnhatn !

tlrx added a commit that referenced this pull request Jan 25, 2019
When an index is frozen, two index settings are updated (index.frozen and 
index.search.throttled) but the settings version is left unchanged and does 
not reflect the settings update. This commit change the 
TransportFreezeIndexAction so that it also increases the settings version 
when an index is frozen/unfrozen.

This issue has been caught while working on the replication of closed 
indices (#3388) in which index metadata for a closed index are updated 
to frozen metadata and this specific assertion tripped.
@tlrx tlrx mentioned this pull request Jan 30, 2019
50 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed/Distributed A catch all label for anything in the Distributed Area. If you aren't sure, use this one. >non-issue v6.7.0 v7.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants