GeorgeJahad commented on PR #4291: URL: https://github.com/apache/ozone/pull/4291#issuecomment-1438919466
> Since the change is mostly done in RequestContext, it should be easy to add support for Ozone native ACL in OzoneNativeAuthorizer as well. FYI @smengcl I believe the OzoneNativeAuthorizer in the master branch already works with snapshots. When I created the OmMetadataReader, I tried to implement it in such a way that ACL's would be read from the snapshot assigned to the OmMetadataReader. I never wrote the tests for it because @prashantpogde asked me to put that work on hold. But those tests are now being written by my team mate: @mladjan-gadzic They appear to be all working, and he will submit the PR in the next few days. Bottom line, I don't believe there are any more changes needed for the native authorizor, just some tests which will be submitted shortly. -- 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]
