Skip to content

Reenable operation-based recoveries for old copies #51380

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
Jan 24, 2020

Conversation

dnhatn
Copy link
Member

@dnhatn dnhatn commented Jan 23, 2020

We accidentally disabled operation-based recoveries for indices with soft-deletes created before 7.6 in #51189.

Relates #51189
Closes #51274

I labelled this non-issue for an unreleased bug.

@dnhatn dnhatn added >non-issue :Distributed Indexing/Recovery Anything around constructing a new shard, either from a local or a remote source. v8.0.0 labels Jan 23, 2020
@dnhatn dnhatn requested a review from ywelsch January 23, 2020 22:16
@elasticmachine
Copy link
Collaborator

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

@dnhatn dnhatn changed the title Enable operation-based recoveries for old copies Reenable operation-based recoveries for old copies Jan 23, 2020
Copy link
Contributor

@ywelsch ywelsch left a comment

Choose a reason for hiding this comment

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

LGTM

@dnhatn
Copy link
Member Author

dnhatn commented Jan 24, 2020

Thanks Yannick.

@dnhatn dnhatn merged commit 80cacc6 into elastic:master Jan 24, 2020
@dnhatn dnhatn deleted the fix-recovery branch January 24, 2020 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed Indexing/Recovery Anything around constructing a new shard, either from a local or a remote source. >non-issue v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CI] BWC failures for :qa:full-cluster-restart testOperationBasedRecovery
4 participants