rdblue commented on code in PR #6622: URL: https://github.com/apache/iceberg/pull/6622#discussion_r1115080821
########## api/src/main/java/org/apache/iceberg/TableScan.java: ########## @@ -101,4 +101,11 @@ default TableScan appendsAfter(long fromSnapshotId) { * @return the Snapshot this scan will use */ Snapshot snapshot(); + + /** + * Create a new {@link TableScan} from this scan's configuration that will have column stats + * + * @return a new scan based on this with column stats + */ + TableScan withColStats(); Review Comment: Rather than adding a revapi change, can you add a default implementation that throws `UnsupportedOperationException`? We want to avoid a breaking API change. -- 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