xanderbailey opened a new pull request, #17791: URL: https://github.com/apache/iceberg/pull/17791
## Before this PR Snapshot expiration fails whenever `gc.enabled` is false, preventing shared-file tables from removing old snapshots even though expiring metadata does not require file deletion. ## After this PR Snapshot expiration removes old snapshot metadata when GC is disabled while skipping all physical file cleanup. This applies consistently to core operations, Spark actions and procedures, and Flink maintenance. ==COMMIT_MSG== allow snapshot expiration when gc is disabled ==COMMIT_MSG== ## Testing Added core unit coverage and Spark action, Spark procedure, and Flink processor tests across the supported versioned sources. Ran the core, Spark 4.1, and Flink 2.1 focused tests, plus `spotlessCheck` and `revapi`. ## Possible downsides? Expired snapshots may leave unreferenced data and metadata files behind. After enabling GC, users must run orphan-file cleanup to remove them. ## Are Docs needed? Yes. Updated the table property, Spark procedure and migration docs, and Nessie guidance. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
