nevzheng opened a new pull request, #12260: URL: https://github.com/apache/gravitino/pull/12260
### What changes were proposed in this pull request? > **Iceberg REST purge stack (Stack 30), review 2 of 8.** Depends on #12259. Files changed are cumulative until earlier stack layers land. Review only commit `8efe3067f`. This PR is published as a draft for design alignment. - Add nullable `manifests_total` and `manifests_done` columns for supported backends. - Deduplicate reachable manifests across snapshots and count completed manifests in memory. - Persist progress only on the existing ownership-fenced heartbeat write. - Reset the estimate after reclaim; never use it to skip or resume work. **After this PR:** operators can observe long-running cleanup without adding checkpoints. ### Why are the changes needed? Large table cleanup can otherwise remain opaque for minutes. The counters are advisory operator information and do not participate in cleanup correctness. Part of #12258. Review order: #12259 → **this PR**. ### Does this PR introduce _any_ user-facing change? No external REST API change. ### How was this patch tested? Tests cover completion, cross-snapshot deduplication, best-effort heartbeat writes, ownership loss, and reset after reclaim. `spotlessJavaCheck`, `TestIcebergCleanupManifestProgress`, `TestIcebergCleanupManager`, and `TestIcebergCleanupJob` pass. -- 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]
