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


##########
api/src/main/java/org/apache/iceberg/TableScan.java:
##########
@@ -148,7 +89,9 @@ default TableScan select(String... columns) {
    * @return a table scan which can read append data from {@code 
fromSnapshotId}
    * exclusive and up to {@code toSnapshotId} inclusive
    */
-  TableScan appendsBetween(long fromSnapshotId, long toSnapshotId);

Review Comment:
   yes, once new IncrementAppendScan is implemented, we can mark these two 
appends methods as deprecated



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