rdblue commented on code in PR #16025:
URL: https://github.com/apache/iceberg/pull/16025#discussion_r4161513743


##########
format/spec.md:
##########
@@ -977,18 +1116,18 @@ For other optional snapshot summary fields, see 
[Appendix F](#optional-snapshot-
 Data and delete files for a snapshot can be stored in more than one manifest. 
This enables:
 
 * Appends can add a new manifest to minimize the amount of data written, 
instead of adding new records by rewriting and appending to an existing 
manifest. (This is called a “fast append”.)
-* Tables can use multiple partition specs. A table’s partition configuration 
can evolve if, for example, its data volume changes. Each manifest uses a 
single partition spec, and queries do not need to change because partition 
filters are derived from data predicates.
+* Tables can use multiple partition specs. A table’s partition configuration 
can evolve if, for example, its data volume changes. Queries do not need to 
change because partition filters are derived from data predicates. In v1-v3, 
each manifest uses a single partition spec.

Review Comment:
   ```suggestion
   * Tables can use multiple partition specs. A table’s partition configuration 
can evolve if, for example, its data volume changes. Partition predicates for a 
partition spec are derived from data predicates and can be applied to filter 
files written using that spec. Prior to v4, a manifest stored files partitioned 
by single spec.
   ```



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