bk-mz commented on PR #9945:
URL: https://github.com/apache/iceberg/pull/9945#issuecomment-2165151682

   This looks plain wrong. 
   
   If I we set shuffle.partititions to 1 this will affect the rewrite 
procedure, it will collapse all delete files into one, meaning we'd be having a 
situation where 1 delete file affects all files of the partitions, which in 
turn would lead to querying all files from it for any query, also damage 
pruning/statistics update.


-- 
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]

Reply via email to