qzyu999 commented on issue #3130: URL: https://github.com/apache/iceberg-python/issues/3130#issuecomment-4468121616
Mentioned in #3131: > As mentioned in https://github.com/apache/iceberg-python/issues/3319, I would like to delay the merging of this PR https://github.com/apache/iceberg-python/pull/3131 until the completion of https://github.com/apache/iceberg-python/pull/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 https://github.com/apache/iceberg-python/pull/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]
