stevenzwu commented on code in PR #4580:
URL: https://github.com/apache/iceberg/pull/4580#discussion_r856499496


##########
api/src/main/java/org/apache/iceberg/Table.java:
##########
@@ -53,6 +53,17 @@ default String name() {
    */
   TableScan newScan();
 
+  /**
+   * Create a new {@link IncrementalAppendScan scan} for this table.
+   * <p>
+   * Once a scan is created, it can be refined to project columns and filter 
data.
+   *
+   * @return an incremental scan for appends only snapshots
+   */
+  default IncrementalAppendScan newIncrementalAppendScan() {

Review Comment:
   @flyrain @aokolnychyi @openinx @chenjunjiedada @yittg @hameizi After 
discussing with @rdblue, we think it is probably cleaner to have separate 
newScan method for appends only and CDC read. 
   
   In the future, we can add `Table#newIncrementalChangelogScan` and 
`IncrementalChangelogScan` interface



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