szehon-ho opened a new pull request #4293:
URL: https://github.com/apache/iceberg/pull/4293


   https://github.com/apache/iceberg/pull/2925 adds isolation level support to 
the core Iceberg API ReplacePartitions, and also exposes it via Spark 
DataFrame.overwritePartitions() API.
   
   This change is to extend isolation level support to the Spark 
DataFrame.overwrite(filter) API for symmetry. The underlying core Iceberg API 
(OverwriteFiles) already supported isolation level validation in this case, so 
the change is smaller.
   
   One observation, DF.overwrite(filter) will be less aggressive than 
DF.overwritePartitions() in concurrent validation due to the two different API 
code paths.  OverwriteFiles checks exactly for the file that will be 
re-written, so in snapshot isolation mode will not throw an exception if 
another file was deleted in the same partition. This is unlike 
ReplacePartitions API which throws exception if any file was deleted in the 
same partition, as it does not keep track of files but rather whole partitions.


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