[GitHub] [ozone] jojochuang commented on a diff in pull request #4234: HDDS-7769. Implement client initiated lease recovery

["jojochuang (via GitHub)" <gi...@apache.org>]

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


##########
hadoop-ozone/interface-storage/src/main/java/org/apache/hadoop/ozone/om/OMMetadataManager.java:
##########
@@ -505,6 +505,19 @@ String getOpenFileName(long volumeId, long bucketId,
    */
   String getRenameKey(String volume, String bucket, long objectID);
 
+  /**
+   * Returns DB key name of an open file in OM metadata store. Should be
+   * #open# prefix followed by actual leaf node name.
+   *
+   * @param volumeId       - ID of the volume
+   * @param bucketId       - ID of the bucket
+   * @param parentObjectId - parent object Id
+   * @param fileName       - file name
+   * @return DB directory key as String.
+   */
+  String getOpenFileNamePrefix(long volumeId, long bucketId,

Review Comment:
   Actually this is a miss on my part. This is no longer needed. Will remove it in the next iteration.



-- 
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-unsubscribe@ozone.apache.org

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


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