chihsuan commented on code in PR #11449:
URL: https://github.com/apache/ozone/pull/11449#discussion_r4246971777


##########
hadoop-ozone/common/src/main/java/org/apache/hadoop/ozone/om/protocol/OzoneManagerProtocol.java:
##########
@@ -622,6 +622,12 @@ default OmMultipartUploadCompleteInfo 
completeMultipartUpload(
         "this to be implemented, as write requests use a new approach.");
   }
 
+  /** Abort a multipart upload, optionally resolving its S3 volume on OM. */

Review Comment:
   nit: Could we mention that OM replaces the volume in `omKeyArgs` when 
`assumeS3Context` is true?
   
   ```suggestion
     /**
      * Abort a multipart upload.
      * @param assumeS3Context if true, OM resolves the S3 volume and ignores 
the volume in omKeyArgs.
      */
   ```



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