manuzhang commented on code in PR #14731:
URL: https://github.com/apache/iceberg/pull/14731#discussion_r2579353386


##########
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:
   If it's not allowed, I think we need stronger words like `must not be`.



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