freesinger commented on PR #11188:
URL: https://github.com/apache/gravitino/pull/11188#issuecomment-4608794125
> Should we remove the event `set owner`, too?
Yes, the set-owner event should also be skipped for Iceberg REST internal
ownership synchronization.
I've already updated the Iceberg REST hook dispatchers to use
internalOwnerDispatcher() for schema/table/view owner sync. The internal owner
dispatcher points directly to OwnerManager, so the owner metadata is still
persisted, but it bypasses OwnerEventManager and avoids emitting an extra
user-visible set-owner event.
--
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]
