wypoon opened a new issue #3070:
URL: https://github.com/apache/iceberg/issues/3070


   From the code in (spark2) `Reader#tasks()` and (spark3) 
`SparkBatchQueryScan#tasks()`, it would appear that if both "snapshot-id" and 
"as-of-timestamp" are specified as options in the `DataFrameReader`, then the 
timestamp takes precedence:
   ```
         if (snapshotId != null) {
           scan = scan.useSnapshot(snapshotId);
         }
   
         if (asOfTimestamp != null) {
           scan = scan.asOfTime(asOfTimestamp);
         }
   ```
   The call to `asOfTime` on the `TableScan` would replace the snapshot set by 
the `useSnapshot`. 
   
   We don't disallow setting both options. However, I think that if the 
"snapshot-id" is set, it should take precedence.
   I am interested to know what the community thinks.


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

Reply via email to