Skip to content

Wipe shard state before switching recovered files live#10179

Merged
s1monw merged 1 commit intoelastic:masterfrom
s1monw:issues/10053
Mar 20, 2015
Merged

Wipe shard state before switching recovered files live#10179
s1monw merged 1 commit intoelastic:masterfrom
s1monw:issues/10053

Conversation

@s1monw
Copy link
Copy Markdown
Contributor

@s1monw s1monw commented Mar 20, 2015

Today we leave the shard state behind even if a recovery is half finished
this causes in rare conditions shards to be recovered and promoted as
primaries that have never been fully recovered.

Closes #10053

@s1monw s1monw added the review label Mar 20, 2015
@s1monw
Copy link
Copy Markdown
Contributor Author

s1monw commented Mar 20, 2015

@bleskes if you have a few moments

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

why do we need this?

@bleskes
Copy link
Copy Markdown
Contributor

bleskes commented Mar 20, 2015

LGTM. Left a minor comment

Today we leave the shard state behind even if a recovery is half finished
this causes in rare conditions shards to be recovered and promoted as
primaries that have never been fully recovered.

Closes elastic#10053
@s1monw s1monw merged commit 93fedcb into elastic:master Mar 20, 2015
@s1monw s1monw deleted the issues/10053 branch March 20, 2015 15:43
@dakrone
Copy link
Copy Markdown
Member

dakrone commented Mar 26, 2015

@s1monw it looks like this was just merged to master, is that correct? Can you label the PR?

@clintongormley clintongormley added >enhancement v2.0.0-beta1 :Distributed/Recovery Anything around constructing a new shard, either from a local or a remote source. labels Apr 5, 2015
@clintongormley clintongormley changed the title [RECOVERY] Wipe shard state before switching recovered files live Wipe shard state before switching recovered files live Jun 7, 2015
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. >enhancement v2.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Recovery should wipe the shard state file before starting recovery

4 participants