Skip to content

Reenable operation-based recoveries for old copies#51380

Merged
dnhatn merged 1 commit intoelastic:masterfrom
dnhatn:fix-recovery
Jan 24, 2020
Merged

Reenable operation-based recoveries for old copies#51380
dnhatn merged 1 commit intoelastic:masterfrom
dnhatn:fix-recovery

Conversation

@dnhatn
Copy link
Copy Markdown
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/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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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/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