kevinjqliu commented on code in PR #14656:
URL: https://github.com/apache/iceberg/pull/14656#discussion_r3210266957
##########
format/spec.md:
##########
@@ -963,24 +969,26 @@ many statistics files associated with different table
snapshots.
Statistics files metadata within `statistics` table metadata field is a struct
with the following fields:
-| v1 | v2 | Field name | Type | Description |
-|----|----|------------|------|-------------|
-| _required_ | _required_ | **`snapshot-id`** | `long` | ID of the Iceberg
table's snapshot the statistics file is associated with. |
-| _required_ | _required_ | **`statistics-path`** | `string` | Path of the
statistics file. See [Puffin file format](puffin-spec.md). |
-| _required_ | _required_ | **`file-size-in-bytes`** | `long` | Size of the
statistics file. |
-| _required_ | _required_ | **`file-footer-size-in-bytes`** | `long` | Total
size of the statistics file's footer (not the footer payload size). See [Puffin
file format](puffin-spec.md) for footer definition. |
-| _optional_ | _optional_ | **`key-metadata`** | Base64-encoded
implementation-specific key metadata for encryption. |
-| _required_ | _required_ | **`blob-metadata`** | `list<blob metadata>` (see
below) | A list of the blob metadata for statistics contained in the file with
structure described below. |
+=== "v1 - v3"
+ | v1 | v2 and v3 | Field name | Type
| Description |
+ | ---------- | ----------
|---------------------------------|-----------------------|-------------|
+ | _required_ | _required_ | **`snapshot-id`** | `long`
| ID of the Iceberg table's snapshot the statistics file is associated
with. |
+ | _required_ | _required_ | **`statistics-path`** | `string`
| Path of the statistics file. See [Puffin file
format](puffin-spec.md). |
+ | _required_ | _required_ | **`file-size-in-bytes`** | `long`
| Size of the statistics file. |
+ | _required_ | _required_ | **`file-footer-size-in-bytes`** | `long`
| Total size of the statistics file's footer (not the footer payload
size). See [Puffin file format](puffin-spec.md) for footer definition. |
+ | _optional_ | _optional_ | **`key-metadata`** |
| Base64-encoded implementation-specific key metadata for encryption. |
+ | _required_ | _required_ | **`blob-metadata`** | `list<blob
metadata>` | A list of the blob metadata for statistics contained in the file
with structure described below. |
Review Comment:
i applied this change since i dont think it was intentional
--
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]
