errose28 commented on a change in pull request #3177:
URL: https://github.com/apache/ozone/pull/3177#discussion_r828306360



##########
File path: 
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/s3/tenant/OMTenantRequestHelper.java
##########
@@ -59,16 +59,17 @@ static void checkAdmin(OzoneManager ozoneManager) throws 
OMException {
   }
 
   /**
-   * Passes check if caller is an Ozone cluster admin or tenant delegated 
admin,
-   * throws OMException otherwise.
+   * Check if caller is an Ozone cluster admin or tenant (delegated) admin.
+   * Throws PERMISSION_DENIED if the check failed.
+   *
    * @throws OMException PERMISSION_DENIED
    */
   static void checkTenantAdmin(OzoneManager ozoneManager, String tenantId)
       throws OMException {
 
     final UserGroupInformation ugi = ProtobufRpcEngine.Server.getRemoteUser();

Review comment:
       See above, not a problem.




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