bshashikant commented on a change in pull request #2172:
URL: https://github.com/apache/hadoop/pull/2172#discussion_r470689196
##########
File path:
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/FSDirRenameOp.java
##########
@@ -821,6 +823,29 @@ void updateQuotasInSourceTree(BlockStoragePolicySuite
bsps) {
}
}
+ private static void checkUnderSameSnapshottableRoot(
+ FSDirectory fsd, INodesInPath srcIIP, INodesInPath dstIIP)
+ throws IOException {
+ // Ensure rename out of a snapshottable root is not permitted if ordered
+ // snapshot deletion feature is enabled
+ SnapshotManager snapshotManager = fsd.getFSNamesystem().
+ getSnapshotManager();
+ if (snapshotManager.isSnapshotDeletionOrdered() && fsd.getFSNamesystem()
+ .isSnapshotTrashRootEnabled()) {
+ INodeDirectory srcRoot = snapshotManager.
+ getSnapshottableAncestorDir(srcIIP);
+ INodeDirectory dstRoot = snapshotManager.
+ getSnapshottableAncestorDir(dstIIP);
+ // Ensure snapshoottable root for both src and dest are same.
+ if (srcRoot != dstRoot) {
Review comment:
We should not use equals here as it is overriden in INODE.java which
just matches the inode id.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
