flyrain commented on code in PR #6350:
URL: https://github.com/apache/iceberg/pull/6350#discussion_r1038666462
##########
spark/v3.3/spark/src/main/java/org/apache/iceberg/spark/SparkReadOptions.java:
##########
@@ -32,6 +32,12 @@ private SparkReadOptions() {}
// End snapshot ID used in incremental scans (inclusive)
public static final String END_SNAPSHOT_ID = "end-snapshot-id";
+ // Start timestamp used in multi-snapshot scans (exclusive)
+ public static final String START_TIMESTAMP = "start-timestamp";
+
+ // End timestamp used in multi-snapshot scans (inclusive)
+ public static final String END_TIMESTAMP = "end-timestamp";
Review Comment:
The other option here is to reuse `as-of-timestamp` as the end timestamp,
but I prefer the symmetric options(start-timestamp, end-timestamp). The UX
would be better this way, I assume.
--
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: issues-unsubscr...@iceberg.apache.org
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
