This is an automated email from the ASF dual-hosted git repository.

shashikant pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/hadoop.git


The following commit(s) were added to refs/heads/trunk by this push:
     new 85d4718  HDFS-15319. Fix INode#isInLatestSnapshot() API. Contributed 
by Shashikant Banerjee.
85d4718 is described below

commit 85d4718ed737d3bfadf815765336465a7a98bc47
Author: Shashikant Banerjee <[email protected]>
AuthorDate: Wed Jul 15 10:32:25 2020 +0530

    HDFS-15319. Fix INode#isInLatestSnapshot() API. Contributed by Shashikant 
Banerjee.
---
 .../src/main/java/org/apache/hadoop/hdfs/server/namenode/INode.java     | 2 --
 1 file changed, 2 deletions(-)

diff --git 
a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/INode.java
 
b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/INode.java
index 6545777..6334ed2 100644
--- 
a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/INode.java
+++ 
b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/INode.java
@@ -254,8 +254,6 @@ public abstract class INode implements INodeAttributes, 
Diff.Element<byte[]> {
     // if parent is a reference node, parent must be a renamed node. We can 
     // stop the check at the reference node.
     if (parent != null && parent.isReference()) {
-      // TODO: Is it a bug to return true?
-      //       Some ancestor nodes may not be in the latest snapshot.
       return true;
     }
     final INodeDirectory parentDir = getParent();


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to