[ 
https://issues.apache.org/jira/browse/HIVE-28102?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17822612#comment-17822612
 ] 

Zoltán Borók-Nagy commented on HIVE-28102:
------------------------------------------

Invoking validateDeletedFiles() as well might also makes sense. 
[https://github.com/apache/iceberg/blob/08e31ce32d506e0dd95e4710ec9e9afba35efa7f/api/src/main/java/org/apache/iceberg/RowDelta.java#L95]

Though if we have validateNoConflictingDataFiles() then we are possibly good 
already (i.e. probably no need for validateDeletedFiles(), but we still need 
validateDataFilesExist()).

> Invoke validateDataFilesExist for RowDelta operations
> -----------------------------------------------------
>
>                 Key: HIVE-28102
>                 URL: https://issues.apache.org/jira/browse/HIVE-28102
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Zoltán Borók-Nagy
>            Assignee: Ayush Saxena
>            Priority: Major
>              Labels: iceberg
>
> Hive must invoke validateDataFilesExist for RowDelta operations 
> (DELETE/UPDATE/MERGE).
> Without this a concurrent RewriteFiles (compaction) and RowDelta can corrupt 
> a table.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to