mehakmeet commented on issue #10713: URL: https://github.com/apache/gravitino/issues/10713#issuecomment-4206289947
I think we agree then on the design change. Currently we fire a failure event for the creation itself if the hook isn't able to set the owner, even tho the object is created by the manager. Now, we will have a success event if the manager succeeds in creating the object. If set owner fails later in post hook operation, that can be logged or rolled back depending on the object. Catalog(already handled), Role, Tag, Policy, Job/JobTemplate and metalake(we should roll back this too?). Thinking maybe we can let the owner be empty then for these cases like we do today if setOwner fails for them and log that error. Schema, Table, Fileset, Topic, Model. What do you think? -- 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]
