danielcweeks opened a new pull request, #6811: URL: https://github.com/apache/iceberg/pull/6811
After discussing with @rdblue about the value of loading refs and snapshots lazily as proposed in #6706, it makes more sense to always load refs and the referenced snapshots, but allow lazy loading of all other snapshots. This PR is a much smaller alternative to #6706 that keeps all snapshot management within the `TableMetadata` class since we're only concerned with snapshots being loaded. The optional snapshot supplier should always load all snapshots of the table if called, which will happen only in the event that: 1. All snapshots need to be traversed (via `TableMetadata::snapshots()`) 2. A specific snapshot is referenced that does not exist in the current snapshot index 3. The TableMetadata is used as the base for building new metadata (via `TableMetadata.buildFrom(...)`) -- 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]
