[GitHub] [iceberg] rdblue commented on a diff in pull request #4945: Add table spec changes for statistics information in table snapshot

[GitBox <gi...@apache.org>]

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


##########
format/spec.md:
##########
@@ -496,6 +496,7 @@ A snapshot consists of the following fields:
 | _optional_ |            | **`manifests`**          | A list of manifest file locations. Must be omitted if `manifest-list` is present |
 | _optional_ | _required_ | **`summary`**            | A string map that summarizes the snapshot changes, including `operation` (see below) |
 | _optional_ | _optional_ | **`schema-id`**          | ID of the table's current schema when the snapshot was created |
+|            | _optional_ | **`statistics`**         | A list of [statistics files' metadata](#statistics-file). The field should be retained by writers, unless writer updates the statistics, or knows they became obsolete. |

Review Comment:
   Why would there be more than one stats file? Wouldn't multiple files be merged together or one would make another obsolete?



-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org