sezruby commented on PR #18603: URL: https://github.com/apache/iceberg/pull/18603#issuecomment-6424961546
@wombatu-kun thanks for the review. All four comments are addressed in b1cb5b05. Could you take another look? @huaxingao could you review this when you have a chance? It narrows UPDATE's check for concurrent delete files to the data files the UPDATE modifies, so conflicts are decided per data file rather than per conflict filter. #17754 is related: it merges concurrently added DVs for DELETE commits on the same data file. The two PRs don't depend on each other. -- 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]
