aokolnychyi commented on code in PR #15360:
URL: https://github.com/apache/iceberg/pull/15360#discussion_r2825439615


##########
spark/v4.1/spark/src/main/java/org/apache/iceberg/spark/source/SparkScanBuilder.java:
##########
@@ -132,68 +98,6 @@ public class SparkScanBuilder
     this(spark, table, null, schema, options);
   }
 
-  private Expression filterExpression() {
-    if (filterExpressions != null) {
-      return filterExpressions.stream().reduce(Expressions.alwaysTrue(), 
Expressions::and);
-    }
-    return Expressions.alwaysTrue();
-  }
-
-  public SparkScanBuilder caseSensitive(boolean isCaseSensitive) {

Review Comment:
   None of the Spark modules have the same guarantees as core or API. Plus, 
this is a brand-new jar for Spark 4.1 that hasn't been released.



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