rdblue commented on a change in pull request #887: Define file and position based deletion file in spec URL: https://github.com/apache/incubator-iceberg/pull/887#discussion_r401750250
########## File path: site/docs/spec.md ########## @@ -254,6 +255,33 @@ Notes: 1. Technically, data files can be deleted when the last snapshot that contains the file as “live” data is garbage collected. But this is harder to detect and requires finding the diff of multiple snapshots. It is easier to track what files are deleted in a snapshot and delete them when that snapshot expires. +#### Deletion Files Review comment: This section should be "Position-based delete files" to distinguish these from equality delete files that will be added later. ---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
