rdblue opened a new pull request, #6399: URL: https://github.com/apache/iceberg/pull/6399
In most catalogs, the `CommitStateUnknownException` is used to signal to `SnapshotProducer` that it is not safe to clean up metadata files because they may have been committed. This introduces another option, "strict cleanup", that will only clean up metadata files if the commit fails with an exception that implements a marker interface, `CleanableFailure`. The new strict mode allows catalogs to default to not cleaning up metadata files, unless the failure is known to be safe -- that the commit state is known. This is useful for the REST catalog, which allows plugging in alternative HTTP libraries. Because exceptions thrown by the REST HTTP client may change, it isn't possible to mark which ones signal that the commit state is unknown. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
