LiJie20190102 commented on PR #13431:
URL: https://github.com/apache/gravitino/pull/13431#issuecomment-6072024085

   > I think we also need to clear all information that is associated with it, 
such as permissions (owner) information, tags, and policies and so on.
   
   Good point. These are already covered by the existing drop path, but the doc 
didn't spell it out. I've updated section 4.1 and added a new Safety 
Consideration #6 to make it explicit:
   
   owner / tag-object links / securable objects / statistics are soft-deleted 
in the same transaction by SchemaMetaService.deleteSchema (and 
TopicMetaService.deleteTopicDependents on the topic side)
   policies detach automatically because they bind to tags, not directly to 
objects
   Ranger privileges are removed by SchemaHookDispatcher → 
authorizationPluginRemovePrivileges
   write-through secrets are removed by 
SecretManager.deleteSecretsFromProperties
   So reconcile-triggered removal behaves exactly like an explicit drop — no 
orphan owner/tag/policy rows are left behind. PTAL.


-- 
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]

Reply via email to