snazy commented on PR #3228: URL: https://github.com/apache/polaris/pull/3228#issuecomment-3641334458
> @snazy : WDYT about holding this PR until some progress is made on #3250 ? No strong objections. However, given that a bunch of code (mostly `Resolver` and the resolution-manifest and all the use/call sites) rely on e.g. `ResolvedPolarisEntity` I'm not sure whether we can go with just one approach. Mean, we do not need entities and with that no "resolved entities" (aka entities plus grant records) in the "external" cases. The idea here was to allow "declarative skipping" of the unnecessary invocations as a short-term workaround before doing a bigger refactoring. -- 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]
