szehon-ho commented on a change in pull request #3284:
URL: https://github.com/apache/iceberg/pull/3284#discussion_r727418411
##########
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:
Shouldn't it be added to the last column? (description)
--
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]
