singhpk234 commented on code in PR #16294:
URL: https://github.com/apache/iceberg/pull/16294#discussion_r3226695427


##########
format/spec.md:
##########
@@ -751,6 +751,8 @@ A snapshot consists of the following fields:
     |            |            | _required_ | **`added-rows`**             | 
The upper bound of the number of rows with assigned row IDs, see [Row 
Lineage](#row-lineage) |
     |            |            | _optional_ | **`key-id`**                 | ID 
of the encryption key that encrypts the manifest list key metadata |
 
+In format version 4 and above, a snapshot's `timestamp-ms` must be strictly 
greater than the `timestamp-ms` of its parent snapshot on the same branch. Root 
snapshots (those without a parent) are not constrained by this requirement.

Review Comment:
   how about tables migrated from v3 to v4 ... with broken timestamps ? do we 
need to say something about them ? 



-- 
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]

Reply via email to