You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@ozone.apache.org by GitBox <gi...@apache.org> on 2022/07/13 01:34:24 UTC

[GitHub] [ozone] smengcl commented on pull request #3576: HDDS-6909. [Multi-Tenant] Use RangerClient for Ranger operations

smengcl commented on PR #3576:
URL: https://github.com/apache/ozone/pull/3576#issuecomment-1182662845

   > Most of my comments are minor nits or future work, the CI is failing in findbugs
   > 
   > ```
   > M D SF: Switch statement found in org.apache.hadoop.ozone.om.multitenant.RangerClientMultiTenantAccessController.decodeRSEStatusCodes(RangerServiceException) where default case is missing  At RangerClientMultiTenantAccessController.java:[lines 152-157]
   > ```
   > 
   > Once it passes we can merge this change in.
   
   Thanks @kerneltime for the review. I have addressed the comments so far.


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