RussellSpitzer commented on code in PR #18526:
URL: https://github.com/apache/iceberg/pull/18526#discussion_r4364639309
##########
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`
| Commit time of the
snapshot that added or last modified the file's data. Inherited when null. See
[Commit Timestamp Inheritance](#commit-timestamp-inheritance). |
Review Comment:
```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]
