nevzheng opened a new pull request, #12262: URL: https://github.com/apache/gravitino/pull/12262
### What changes were proposed in this pull request? This is Stack 30, PR 4 of 8. - Verify that J1 is `RUNNING`, D1 is `PURGING` and owned by J1, and the exact `table_id`, `deletion_id`, and immutable parent route match. - Perform metadata-only `dropTable` on the reserved table route. - Treat an already-missing registration as idempotent success. - Fail the cleanup attempt on any other provider error before deleting files. - Preserve the existing unlinked V1 job behavior. After this PR, a V2 worker removes the validated retained registration before physical cleanup begins. ### Why are the changes needed? Physical files must not be deleted while their retained Iceberg registration remains. The reserved Gravitino route is the POC ownership boundary. Part of #12258. ### Does this PR introduce _any_ user-facing change? No new endpoint. Retained table names remain hidden and reserved. ### How was this patch tested? Test coverage is included for direct registration removal, already-missing registrations, exact relational linkage, parent routing, worker ordering, missing cleaner configuration, and sanitized provider failures. No post-rebase local test run was performed; CI validation is pending. -- 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]
