edwinchoi commented on pull request #1508: URL: https://github.com/apache/iceberg/pull/1508#issuecomment-712654972
@rdblue what about adding a method `Table rewind(Snapshot)` to `Table`? This seems like a more straight-forward way to handle views as-of a snapshot or moment in time with potentially enough flexibility to work with v2 (queries as-of a moment would select the snapshot upstream). The return value would be a new instance that prohibits any mutating operations against `Table` and `TableOperations`. I suspect there would need to be an equivalent `rewind` on `TableOperations` as well for fetching the matching metadata file (there are a lot of fields in the metadata object that seem relevant to queries, e.g., partition specs, table properties, and sort orders.. I'm assuming it's necessary to know these to read a historic snapshot, and not simply the schema alone). ---------------------------------------------------------------- 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
