wypoon commented on code in PR #14048:
URL: https://github.com/apache/iceberg/pull/14048#discussion_r2342437550
##########
format/spec.md:
##########
@@ -754,9 +754,9 @@ A snapshot consists of the following fields:
| _optional_ | _required_ | _required_ | **`summary`** | A
string map that summarizes the snapshot changes, including `operation` as a
_required_ field (see below) |
| _optional_ | _optional_ | _optional_ | **`schema-id`** | ID of
the table's current schema when the snapshot was created
|
| | | _required_ | **`first-row-id`** | The
first `_row_id` assigned to the first row in the first data file in the first
manifest, see [Row Lineage](#row-lineage) |
+| | | _required_ | **`added-rows`** | The upper
bound of the number of rows with assigned row IDs, see [Row
Lineage](#row-lineage) |
Review Comment:
Nit: It seems that the table is currently formatted so that the widths of
the column line up. Can we make the Field (`added-rows`) column here the same
width as in the other rows?
--
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]
