lawofcycles commented on PR #3320: URL: https://github.com/apache/iceberg-python/pull/3320#issuecomment-4723852189
@rambleraptor Pushed a fix for multi-producer retry. When `Transaction.delete()` creates both `_DeleteFiles` and `_OverwriteFiles`, the second producer's `_parent_snapshot_id` points to the first producer's uncommitted snapshot on retry, so validation fails because that snapshot doesn't exist in the catalog. Fixed by introducing `CommitWindow`, a dataclass that carries `starting_snapshot_id` and `catalog_head_snapshot_id` separately from `_parent_snapshot_id`. `_rebuild_snapshot_updates` builds one from the actual catalog state and shares it across all producers. Added `test_mixed_delete_overwrite_retries_successfully` to cover this path. -- 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]
