swamirishi commented on code in PR #8555:
URL: https://github.com/apache/ozone/pull/8555#discussion_r2138778730


##########
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/OmSnapshotManager.java:
##########
@@ -764,6 +764,11 @@ public static String getSnapshotPath(OzoneConfiguration 
conf,
         OM_DB_NAME + snapshotInfo.getCheckpointDirName();
   }
 
+  public static String getSnapshotLocalPropertyYamlPath(OzoneConfiguration 
conf,
+      SnapshotInfo snapshotInfo) {
+    return getSnapshotPath(conf, snapshotInfo) + ".yaml";

Review Comment:
   yeah increasing the number of directories can just confuse people



-- 
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: issues-unsubscr...@ozone.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@ozone.apache.org
For additional commands, e-mail: issues-h...@ozone.apache.org

Reply via email to