nssalian commented on code in PR #2302: URL: https://github.com/apache/iceberg-go/pull/2302#discussion_r4404981007
########## table/rewrite_data_files.go: ########## Review Comment: nit: this still says "groups whose catalog commits failed", but groups rejected by the new pre-check never reach the catalog. Maybe "groups whose batch failed to commit or was rejected because a delete for one of its files was committed after planning" -- 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]
