kbendick commented on a change in pull request #3284:
URL: https://github.com/apache/iceberg/pull/3284#discussion_r727452321
##########
File path: site/docs/spec.md
##########
@@ -379,7 +379,7 @@ The schema of a manifest file is a struct called
`manifest_entry` with the follo
| _required_ | _required_ | **`100 file_path`** | `string`
| Full URI for the file with FS scheme |
| _required_ | _required_ | **`101 file_format`** | `string`
| String file format name, avro, orc or parquet |
| _required_ | _required_ | **`102 partition`** | `struct<...>`
| Partition data tuple, schema based on the partition spec output
using partition field ids for the struct field ids |
-| _required_ | _required_ | **`103 record_count`** | `long`
| Number of records in this file |
+| _required_ | _required_ | **`103 record_count`** | `long` with
special value: `-1: Record count unknown` | Number of records in this file. |
Review comment:
But since this is just a caveat on one magic value, I do tend to agree
with you.
I'm open to either. Given it's the spec, I figured I'd let others weigh in.
--
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-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
