rdsr edited a comment on issue #1109:
URL: https://github.com/apache/iceberg/issues/1109#issuecomment-642917053


   Also, the latest lastUpdatedMillis can be lower from the metadata log entry 
timestamps. That should also be checked.   
   
   In our case the clock skew was around 3 secs, so 1 sec may be too small a 
value. Would you consider making this configurable?
   
   Side question, if we always check that the lastUpdatedMillis field is >= the 
latest previous log entry with a tolerance,  We don't need to check all the log 
entries every time we create a TableMetadata instance
   


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

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to