rdblue commented on code in PR #16025: URL: https://github.com/apache/iceberg/pull/16025#discussion_r4161635917
########## format/spec.md: ########## @@ -1071,7 +1212,8 @@ Duplicate live manifest entries for the same content file violate [content file Delete files and deletion vector metadata that match the filters must be applied to data files at read time, limited by the following scope rules. -* A deletion vector must be applied to a data file when all of the following are true: +* In v4, a deletion vector must be applied to the data file tracked by the same entry. No path, sequence number, or partition comparison applies because the vector is colocated with the data file. Review Comment: We should also remove "v4" and rely on having a co-located DV. If we have a co-located DV, then it must be used. Also, you are guaranteed that there is no other DV or position delete file for the data file. -- 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]
