jojochuang commented on code in PR #9084:
URL: https://github.com/apache/ozone/pull/9084#discussion_r2399470504


##########
hadoop-ozone/interface-storage/src/main/java/org/apache/hadoop/ozone/om/OMMetadataManager.java:
##########
@@ -675,4 +685,43 @@ String getMultipartKey(long volumeId, long bucketId,
    */
   boolean containsIncompleteMPUs(String volume, String bucket)
       throws IOException;
+
+  /**
+   * Represents a unique identifier for a specific bucket within a volume.
+   *
+   * This class combines a volume identifier and a bucket identifier
+   * to uniquely identify a bucket within a storage system.
+   */
+  class VolumeBucketId {
+    private final long volumeId;

Review Comment:
   @swamirishi I'm confused here. I see this PR went through several iterations 
but the latest version only contains two files OMMetadataManager and 
OMDirectoriesPurgeRequestWithFSO. And it adds bucketId that is not used 
anywhere.



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