szehon-ho commented on code in PR #7105:
URL: https://github.com/apache/iceberg/pull/7105#discussion_r1319942204
##########
format/spec.md:
##########
@@ -702,6 +703,49 @@ Blob metadata is a struct with the following fields:
| _optional_ | _optional_ | **`properties`** | `map<string, string>` |
Additional properties associated with the statistic. Subset of Blob properties
in the Puffin file. |
+#### Partition statistics
+
+Partition statistics files are based on [Partition Statistics file
spec](#partition-statistics-file).
+Partition statistics are not required for reading or planning and readers may
ignore them.
+Each table snapshot may be associated with at most one partition statistic
file.
+A writer can optionally write the partition statistics file during each write
operation. If the statistics file is written for the specific snapshot,
Review Comment:
Just my opinion, I am not too sure these two sentences add much value, it is
the case for any file reference in Iceberg , isnt it?
```A writer can optionally write the partition statistics file during each
write operation. If the statistics file is written for the specific snapshot,
it must be registered in the table metadata file to be considered as a valid
statistics file for the reader.```
--
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]
