adoroszlai commented on PR #7238: URL: https://github.com/apache/ozone/pull/7238#issuecomment-2376121606
Thanks @fapifta for catching this. > * change OzoneAccessAuthorizer to be a native authorizer, this may have some interesting consequences that I do not see, but in general it seems to be ok I vote for this option. `OzoneAccessAuthorizer` is just a no-op (allows everything) and is native to Ozone (vs. being external). > * revisit the restart method We should do this in a follow-up. `restart` is only used in tests. I think such custom logic in tests reduces their ability to catch real bugs, and introduces test-only problems like this. -- 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]
