mrutunjay-kinagi commented on PR #3062: URL: https://github.com/apache/iceberg-python/pull/3062#issuecomment-3924977228
@Fokko Thanks, good suggestion. I pushed a follow-up commit that adds metadata-level timestamp validation in so we validate on metadata construction as well:\n\n- snapshot-log entries must remain sorted (with the same 1-minute skew tolerance used in Java)\n- cannot be more than 1 minute behind the latest snapshot-log entry\n\nI also added parser-level tests in for both checks.\n\nI kept the existing guard too, because that check depends on the previous metadata state during update application (which metadata parsing alone can’t fully recover). -- 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]
