freesinger commented on PR #11188:
URL: https://github.com/apache/gravitino/pull/11188#issuecomment-4608799082
> Do other dispatchers have similar issues? Could u fix them in another pull
request?
I checked similar internal dispatcher usages. Within the Iceberg REST scope,
the relevant paths are catalog lookup, schema/table/view metadata import, and
ownership sync. This PR now uses internal dispatchers for these paths.
There are still some ordinary dispatcher calls outside this scope, for
example in `AuthorizationUtils/Ranger` helper logic and
`TopicOperationDispatcher` schema lookups. I suggest handling those in a
separate follow-up PR if we want to generalize the internal-dispatcher pattern,
because they may affect authorization plugin behavior and non-Iceberg API event
semantics.
--
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]
