danielcweeks commented on code in PR #15630:
URL: https://github.com/apache/iceberg/pull/15630#discussion_r3276737770
##########
format/spec.md:
##########
@@ -954,6 +1012,34 @@ Table metadata consists of the following fields:
| _optional_ | _optional_ | _optional_ | **`partition-statistics`** | A
list (optional) of [partition statistics](#partition-statistics). |
| | | _required_ | **`next-row-id`** | A
`long` higher than all assigned row IDs; the next snapshot’s `first-row-id`.
See [Row Lineage](#row-lineage). |
| | | _optional_ | **`encryption-keys`** | A
list (optional) of [encryption keys](#encryption-keys) used for table
encryption. |
+=== "v4"
+ | v4 | Field | Description |
+ |------------|-----------------------------|-------------|
+ | _required_ | **`format-version`** | An integer version number for
the format. Implementations must throw an exception if a table's version is
higher than the supported version. |
+ | _required_ | **`table-uuid`** | A UUID that identifies the
table, generated when the table is created. Implementations must throw an
exception if a table's UUID does not match the expected UUID after refreshing
metadata. |
+ | _optional_ | **`location`** | The table's base location.
This is used by writers to determine where to store data files, manifest files,
and table metadata files. Must be an absolute path when present. See [Table
Locations](#table-location-specification). |
+ | _required_ | **`last-sequence-number`** | The table's highest assigned
sequence number, a monotonically increasing long that tracks the order of
snapshots in a table. |
+ | _required_ | **`last-updated-ms`** | Timestamp in milliseconds
from the unix epoch when the table was last updated. Each table metadata file
should update this field just before writing. |
+ | _required_ | **`last-column-id`** | An integer; the highest
assigned column ID for the table. This is used to ensure columns are always
assigned an unused ID when evolving schemas. |
+ | | **`schema`** | The table’s current schema.
(**Deprecated**: use `schemas` and `current-schema-id` instead) |
Review Comment:
These fields are actually from the original table carried over. I don't
think we should update the wording here (it's not technically part of this spec
change).
--
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]
