lasdf1234 commented on PR #11828: URL: https://github.com/apache/gravitino/pull/11828#issuecomment-4849889956
There are two issues that need to be noted: 1、In IcebergViewHookDispatcher.registerView, skip import/setOwner when the view entity already exists in Gravitino EntityStore (same rationale as registerTable hooks). 2、Consider failing register when import into Gravitino fails, instead of best-effort warn-only importView, since registration's purpose is to register the entity in Gravitino. -- 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]
