Skip to content

Conversation

@imotov
Copy link
Contributor

@imotov imotov commented Apr 13, 2014

Currently even if some shards of the snapshot are not snapshotted successfully, the snapshot is still marked as "SUCCESS". Users may miss the fact the there are shard failures present in the snapshot and think that snapshot was completed. This change adds a new snapshot state "PARTIAL" that provides a quick indication that the snapshot was only partially successful.

Closes #5792

Copy link
Contributor

Choose a reason for hiding this comment

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

;) thanks :)

@s1monw
Copy link
Contributor

s1monw commented May 15, 2014

I left one comment but this LGTM

@s1monw s1monw removed the review label May 15, 2014
Currently even if some shards of the snapshot are not snapshotted successfully, the snapshot is still marked as "SUCCESS". Users may miss the fact the there are shard failures present in the snapshot and think that snapshot was completed. This change adds a new snapshot state "PARTIAL" that provides a quick indication that the snapshot was only partially successful.

Closes elastic#5792
@imotov imotov merged commit 7f5befd into elastic:master May 16, 2014
@clintongormley clintongormley added >breaking v2.0.0-beta1 :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs labels Aug 13, 2015
@imotov imotov deleted the issue-5792-partial-snapshot-status branch May 1, 2020 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add "PARTIAL" snapshot status

4 participants