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

Add engine failure on recovery finalization corruption back #11241

Merged
merged 1 commit into from May 20, 2015

Conversation

s1monw
Copy link
Contributor

@s1monw s1monw commented May 19, 2015

This engine failure on finalization corruption was lost on refactorings and
should be added back.

This engine failure on finalization corruption was lost on refactorings and
should be added back.
@s1monw s1monw added v2.0.0-beta1 review :Distributed/Recovery Anything around constructing a new shard, either from a local or a remote source. labels May 19, 2015
@s1monw
Copy link
Contributor Author

s1monw commented May 19, 2015

@bleskes can you take a look if you have a sec

@bleskes
Copy link
Contributor

bleskes commented May 20, 2015

Good catch, LGTM.

s1monw added a commit that referenced this pull request May 20, 2015
[RECOVERY] Add engine failure on recovery finalization corruption back
@s1monw s1monw merged commit 40bd56b into elastic:master May 20, 2015
@s1monw s1monw removed the review label May 20, 2015
@s1monw s1monw deleted the readd_fail_on_recovery branch May 20, 2015 12:35
@s1monw s1monw added review and removed review labels May 20, 2015
@clintongormley clintongormley changed the title [RECOVERY] Add engine failure on recovery finalization corruption back Add engine failure on recovery finalization corruption back Jun 8, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug :Distributed/Recovery Anything around constructing a new shard, either from a local or a remote source. v2.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants