laserninja commented on PR #11828: URL: https://github.com/apache/gravitino/pull/11828#issuecomment-4850236339
Thanks for the review @lasdf1234! Both points addressed in dedb8ee: Fail hard on import: Added a new importViewForRegister() method that propagates exceptions instead of using the best-effort importView(). This matches the registerTable pattern in IcebergNamespaceHookDispatcher. Skip if entity already exists: importViewForRegister() checks EntityStore.exists() before importing/setting ownership. If the view entity is already in the Gravitino store, it returns early with a log message. -- 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]
