qzyu999 commented on PR #3131: URL: https://github.com/apache/iceberg-python/pull/3131#issuecomment-4468114397
As mentioned in #3319, I would like to delay the merging of this PR #3131 until the completion of #3320. The reason is that there is a `_validate_concurrency()` function that is being developed there, and upon completion it would make sense to integrate it `_RewriteFiles` to run all conflict validations before trying to commit. It is technically possible to do this currently, but upon completion of #3320 it would make sense to refactor `_RewriteFiles` to utilize the `_validate_concurrency()` function. -- 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]
