[GitHub] [iceberg] wypoon commented on pull request #1508: Use schema at the time of the snapshot when reading a snapshot.

[GitBox <gi...@apache.org>]

wypoon commented on pull request #1508:
URL: https://github.com/apache/iceberg/pull/1508#issuecomment-746454110


   Sorry for the long pause.
   My understanding is that the metadata-log (`TableMetadata.previousFiles()`) is in order of metadata committed, even if the timestamps may not be in order due to clock skew on different machines from which the commits are done. I have implemented @edwinchoi's suggestion to read each metadata file listed in the metadata-log to find the first one where the current-snapshot-id matches the target snapshot id.
   I have rebased on master (probably behind again by now), and made some changes accordingly.
   @rdblue, can you please take a look at this again?


----------------------------------------------------------------
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:
users@infra.apache.org



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