sokui commented on code in PR #7700:
URL: https://github.com/apache/ozone/pull/7700#discussion_r1921878453


##########
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/OmMetadataManagerImpl.java:
##########
@@ -899,11 +898,13 @@ public String getMultipartKeyFSO(String volume, String 
bucket, String key, Strin
     final long volumeId = getVolumeId(volume);
     final long bucketId = getBucketId(volume,
             bucket);
-    long parentId =
-            OMFileRequest.getParentID(volumeId, bucketId, key, this);
-
-    String fileName = OzoneFSUtils.getFileName(key);
+    final String nonFSOMultipartKey =
+        getMultipartKey(volume, bucket, key, uploadId);

Review Comment:
   I think we are talking about different problems. Here is about the 
multipartKey for FSO or non-FSO. The non-FSO multipartKey is of this format: 
`volume/bucket/key/uploadId`, while the FSO one is: 
`volumeId/bucketId/parentId/fileName/uploadId`.
   
   But anyway, I will just change it to `multipartKey` for simplicity.



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