aokolnychyi commented on a change in pull request #3661:
URL: https://github.com/apache/iceberg/pull/3661#discussion_r769732026



##########
File path: 
spark/v3.2/spark/src/main/java/org/apache/iceberg/spark/source/SparkScanBuilder.java
##########
@@ -164,9 +161,45 @@ public Scan build() {
         spark, table, readConf, schemaWithMetadataColumns(), 
filterExpressions);
   }
 
-  public Scan buildMergeScan() {
-    return new SparkMergeScan(
-        spark, table, readConf, ignoreResiduals,
-        schemaWithMetadataColumns(), filterExpressions);
+  public Scan buildCopyOnWriteScan() {

Review comment:
       I moved the construction of `TableScan` here. I'll do the same for 
regular scans in the next PR.
   Merge-on-read will use the existing scan for batch queries (with runtime 
filtering and everything).




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