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

Prevent closing index during snapshot restore #16933

Merged
merged 1 commit into from Mar 8, 2016

Conversation

Projects
None yet
2 participants
@ywelsch
Copy link
Contributor

commented Mar 3, 2016

Closing an index that is being restored from a snapshot should fail the close operation. The previous behavior would close the index and the snapshot process would continue running indefinitely.

Relates to #16321

}
}

public void waitForBlockOnAnyDataNode(String repository, TimeValue timeout) throws InterruptedException {

This comment has been minimized.

Copy link
@imotov

imotov Mar 7, 2016

Member

Why not use awaitBusy here?

This comment has been minimized.

Copy link
@ywelsch

ywelsch Mar 8, 2016

Author Contributor

good idea. I have switched to awaitBusy here.

@imotov

This comment has been minimized.

Copy link
Member

commented Mar 7, 2016

Left a minor comment. Otherwise, LGTM.

@ywelsch ywelsch force-pushed the ywelsch:fix/snaprestore-fails-close branch to 71ac12f Mar 8, 2016

ywelsch added a commit that referenced this pull request Mar 8, 2016

Merge pull request #16933 from ywelsch/fix/snaprestore-fails-close
Prevent closing index during snapshot restore

@ywelsch ywelsch merged commit 20f5255 into elastic:master Mar 8, 2016

1 check passed

CLA Commit author is a member of Elasticsearch
Details

ywelsch added a commit that referenced this pull request Mar 8, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.