syun64 commented on PR #804: URL: https://github.com/apache/iceberg-python/pull/804#issuecomment-2159128257
> This will increase the number of conflicts while there shouldn't be an issue with adding a snapshot and updating the sort order at the same time. My impression was that AssertTableUUID condition won't lead to a conflict in that case, but that it will only fail if a table was dropped and created at the same time. The table UUID seems to be assigned only when a new table is [created](https://github.com/apache/iceberg-python/blob/94e8a9835995e3b61f07f0dfb48d8a22a1e1d1b0/pyiceberg/table/__init__.py#L527) - so commits that add a snapshot or update the sort order should both be operating on the same UUID -- 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
