aokolnychyi commented on a change in pull request #351: Extend Iceberg with a
way to overwrite files for eager updates/deletes
URL: https://github.com/apache/incubator-iceberg/pull/351#discussion_r314234844
##########
File path: api/src/main/java/org/apache/iceberg/OverwriteFiles.java
##########
@@ -74,5 +90,22 @@
*
* @return this for method chaining
*/
- OverwriteFiles validateAddedFiles();
+ OverwriteFiles validateAddedFilesMatchRowFilter();
+
+ /**
+ * Enables validation of files that are added concurrently.
+ * <p>
+ * This method should be called when the table is queried to determine files
to delete/append.
+ * If a concurrent operation commits a new file after the data was read and
that file might
+ * contain rows matching the specified row filter, the overwrite operation
will detect
+ * this during retries and fail.
+ * <p>
+ * Calling this method with a correct row filter is required to maintain
serializable isolation.
+ * Otherwise, the isolation level will be snapshot isolation.
+ *
+ * @param readSnapshotId the snapshot id that was used to read the data
+ * @param rowFilter an expression on rows in the table
+ * @return this for method chaining
+ */
+ OverwriteFiles validateNoConflictingAppends(Long readSnapshotId, Expression
rowFilter);
Review comment:
Allowing `null` was intentional to handle cases when the table is empty.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
