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
HDFS-15817. Rename snapshots while marking them deleted. #2677
Conversation
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
Appending a timestamp to the deleted snapshot name may lead to different names being generated pre and post restart of namenode during edit log replay. Therefore, the idea here is to just append the snapshot Id which will remain constant. @szetszwo , can you have a look. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are a few whitespace changes. Otherwise, the change look good.
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
@bshashikant , the commit 99efed3 did not fix any whitespace changes. The changes might be done by your IDE. I suggest to use VI or other editors to revert those changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes generally look good. However, if a user again deletes a deleted snapshot with the new name(which was given because of this rename. Can we lead into concats of the snapid ?
Yes, it will append the snap id. The behaviour will be same if user for example, creates a snapshot with the same name as snapshot format and deletes it . I guess, this should be fine as marked for deleted snapshots are user visible and can be deleted again and again until and unless it gets permanently deleted. |
💔 -1 overall
This message was automatically generated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 the change looks good.
💔 -1 overall
This message was automatically generated. |
Thanks @mukul1987 and @bshashikant for the review. |
(cherry picked from commit 2df2dfb Ref: OPSAPS-59302. Snapshot based replication doesn't work with ordered snapshot deletion feature turned on Change-Id: I037fbc24f71150c26f90eb12a41073e2a9e0820c
Please see https://issues.apache.org/jira/browse/HDFS-15817 for details.