gaborkaszab commented on code in PR #16425: URL: https://github.com/apache/iceberg/pull/16425#discussion_r4452613062
########## format/spec.md: ########## @@ -157,6 +157,7 @@ Version 4 of the Iceberg spec adds support for relative locations in metadata, e * **Delete manifest** -- A file that lists delete files to be associated with data files at planning time. * **Data file** -- A file that contains rows of a table. * **Delete file** -- A file that encodes rows of a table that are deleted by position or data values. +* **Column file** -- (v4+) A file that stores the values of one or more fields of a data file, or a v4 leaf manifest. Review Comment: Reading this one more time: This might suggest that the fields in a column file have to overlap with the fields in the data file. With a column backfill, this is not the case. If it's only me reading it that way, then I'm fine leaving as is. If not, would you mind rewriting this to be more general covering non-overlapping columns with 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]
