pvary commented on code in PR #14731: URL: https://github.com/apache/iceberg/pull/14731#discussion_r2580355544
########## format/spec.md: ########## @@ -109,6 +109,8 @@ There are two types of row-level deletes: Like data files, delete files are tracked by partition. In general, a delete file must be applied to older data files with the same partition; see [Scan Planning](#scan-planning) for details. Column metrics can be used to determine whether a delete file's rows overlap the contents of a data file or a scan range. +A deleted row should not be undeleted by rewriting the position delete file or delete vector and unsetting the position or bit. Rows can only be added to a table via new data files. Review Comment: Maybe something like: ``` A deleted row must not be restored by rewriting the position delete file or delete vector and clearing the associated position or bit. Rows must only be added to a table by writing new data files. ``` -- 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]
