huaxingao commented on code in PR #18526:
URL: https://github.com/apache/iceberg/pull/18526#discussion_r4383716347
##########
format/spec.md:
##########
@@ -799,6 +803,7 @@ The contents of a file are part of a snapshot if its
tracking `status` is **live
| _optional_ | 5 | **`modified_snapshot_id`** | `long`
| Snapshot ID where the file
was last modified. |
| _optional_ | 3 | **`sequence_number`** | `long`
| Data sequence number of
the file. Inherited when null. See [Sequence Number
Inheritance](#sequence-number-inheritance). |
| _optional_ | 4 | **`file_sequence_number`** | `long`
| File sequence number
indicating when the file was added. Inherited when null. See [Sequence Number
Inheritance](#sequence-number-inheritance). |
+| _optional_ | 8 | **`snapshot_timestamp`** | `timestamptz` | `timestamp-ms`
of the snapshot that added or last modified the file's data, in microseconds.
Inherited when null. See [Snapshot Timestamp
Inheritance](#snapshot-timestamp-inheritance). |
Review Comment:
This row is missing the 4 leading spaces that the other rows in this table
have, so it will break the table rendering (the table is inside a list). The
suggestion also aligns the columns with the other rows.
```suggestion
| _optional_ | 8 | **`snapshot_timestamp`** | `timestamptz`
| `timestamp-ms` of the
snapshot that added or last modified the file's data, in microseconds.
Inherited when null. See [Snapshot Timestamp
Inheritance](#snapshot-timestamp-inheritance). |
```
--
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]
