huaxingao commented on code in PR #17783: URL: https://github.com/apache/iceberg/pull/17783#discussion_r3848479920
########## format/spec.md: ########## @@ -1913,6 +1916,14 @@ Reading v4 metadata: * Relative paths must be resolved against the table location before use (see [Path Resolution](#path-resolution)) * When `location` is omitted, the table location must be provided (see [Table Location Specification](#table-location-specification)) +Row-level delete changes: + +* Equality delete files must not be added to v4 tables +* Readers must continue to apply equality deletes for v2 and v3 tables and for equality deletes carried over into upgraded v4 tables +* Upgrading a v2 or v3 table to v4 is metadata-only and does not rewrite data or delete files + * Equality deletes remain in the carried-over v2 or v3 delete manifests that already track them + * Converting equality deletes to DVs is a separate, optional maintenance action Review Comment: Removed. Thanks! ########## format/spec.md: ########## @@ -1352,13 +1353,13 @@ Notes: ### Delete Formats -This section details how to encode row-level deletes in Iceberg delete files. Row-level deletes are added by v2 and are not supported in v1. Deletion vectors are added in v3 and are not supported in v2 or earlier. Position delete files must not be added to v3 tables, but existing position delete files are valid. +This section details how to encode row-level deletes in Iceberg delete files. Row-level deletes are added by v2 and are not supported in v1. Deletion vectors are added in v3 and are not supported in v2 or earlier. Position delete files must not be added to v3 tables, but existing position delete files are valid. Equality delete files must not be added to v4 tables, but existing equality delete files are valid. There are different formats for encoding row-level deletes: * Deletion vectors (DVs) identify deleted rows within a single referenced data file by position in a bitmap * Position delete files identify deleted rows by file location and row position (**deprecated** in v3) -* Equality delete files identify deleted rows by the value of one or more columns +* Equality delete files identify deleted rows by the value of one or more columns (**deprecated** in v4) Review Comment: Done. -- 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]
