ArafatKhan2198 commented on code in PR #6492:
URL: https://github.com/apache/ozone/pull/6492#discussion_r1588851458


##########
hadoop-ozone/recon/src/main/java/org/apache/hadoop/ozone/recon/tasks/NSSummaryTaskDbEventHandler.java:
##########
@@ -132,6 +132,10 @@ protected void handlePutDirEvent(OmDirectoryInfo 
directoryInfo,
       curNSSummary = new NSSummary();
     }
     curNSSummary.setDirName(dirName);
+    // Set the parent directory ID
+    if (parentObjectId != -1) {
+      curNSSummary.setParentId(parentObjectId);
+    }

Review Comment:
   I have removed the -1 check from there, I do not see any scenario where the 
parentID of any directory would be -1 which does not happen anytime, even if 
that parent is a bucket it will still have a parentID I have double checked it 
by running the JVM debugger on the cluster code.



-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


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

Reply via email to