adutra commented on PR #3228: URL: https://github.com/apache/polaris/pull/3228#issuecomment-3660935336
> 2. What are your thoughts on an alternate approach of pushing the `Supplier`s of these entities into the `PolarisAuthorizer` for lazy evaluation instead, as opposed to exposing public boolean flags? This way, we'd remove a conditional evaluation within the Handlers, and instead push the responsibility of deciding what context is needed and whether we should fetch it only into the `PolarisAuthorizer` Completely agree with you here @sungwy! The `Resolver` is largely an implementation detail of the default RBAC engine. Merging the former into the latter is the right way to go. -- 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]
