xndai commented on PR #17360:
URL: https://github.com/apache/iceberg/pull/17360#issuecomment-5079399546

   > we are trying to fix this with having monotonic timestamps of snapshot in 
v4
   > 
   > https://github.com/apache/iceberg/pull/16294/changes
   > 
   > Can we trust these timestamps in the first place as there is no 
requirement of true time semantics in first place, so i wonder if its worth 
fixing the old behaviour vs just saying hey v4 is when its gonna get fixed ?
   
   Thanks for letting me know that. I'd say it still worth fixing as we still 
have v1, v2 and v3 tables out there. Once v4 becomes official, we could add a 
short circuit break in this loop for v4 as we can assume there are strictly 
ordered. 


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