xBis7 commented on code in PR #4134:
URL: https://github.com/apache/ozone/pull/4134#discussion_r1087745291
##########
hadoop-ozone/ozonefs-common/src/main/java/org/apache/hadoop/fs/ozone/BasicRootedOzoneClientAdapterImpl.java:
##########
@@ -643,6 +645,13 @@ public FileStatusAdapter getFileStatus(String path, URI
uri,
}
try {
OzoneBucket bucket = getBucket(ofsPath, false);
+
+ if (ofsPath.isSnapshotIndicator()) {
Review Comment:
@hemantk-12 I get that you want to get rid of nesting. I don't understand
what you mean to do with the try-catch. Do you want me to wrap all the `if -
else if` block with the try-catch or just wrap whatever line might throw an
exception? Or do you want me to remove the try-catch altogether?
--
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]
