rdblue commented on a change in pull request #3132:
URL: https://github.com/apache/iceberg/pull/3132#discussion_r725392803
##########
File path: spark/src/main/java/org/apache/iceberg/spark/SparkReadOptions.java
##########
@@ -30,6 +30,12 @@ private SparkReadOptions() {
// Snapshot ID of the table snapshot to read
public static final String SNAPSHOT_ID = "snapshot-id";
+ // Start snapshot ID used in incremental scans (exclusive)
+ public static final String START_SNAPSHOT_ID = "start-snapshot-id";
+
+ // End snapshot ID used in incremental scans (inclusive)
+ public static final String END_SNAPSHOT_ID = "end-snapshot-id";
Review comment:
I would probably still add them to the conf parser to be consistent.
--
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]
