This is an automated email from the ASF dual-hosted git repository.

agrove pushed a commit to branch fix/spark-343-filter-pushdown-tag
in repository https://gitbox.apache.org/repos/asf/datafusion-comet.git


The following commit(s) were added to 
refs/heads/fix/spark-343-filter-pushdown-tag by this push:
     new 209885ce2 fix: add tracking issue link to IgnoreCometNativeScan for 
StringPredicate test
209885ce2 is described below

commit 209885ce2ecec1269e93ac9134be1d418a7ad9a6
Author: Andy Grove <[email protected]>
AuthorDate: Wed Apr 1 04:50:01 2026 -0600

    fix: add tracking issue link to IgnoreCometNativeScan for StringPredicate 
test
    
    Address review feedback by including issue #3321 link in the
    IgnoreCometNativeScan reason string so the gap can be tracked.
---
 dev/diffs/3.4.3.diff | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/dev/diffs/3.4.3.diff b/dev/diffs/3.4.3.diff
index 7e98a8f8c..131b88296 100644
--- a/dev/diffs/3.4.3.diff
+++ b/dev/diffs/3.4.3.diff
@@ -2003,7 +2003,7 @@ index 07e2849ce6f..3e73645b638 100644
        ParquetOutputFormat.WRITER_VERSION -> 
ParquetProperties.WriterVersion.PARQUET_2_0.toString
      )
 diff --git 
a/sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFilterSuite.scala
 
b/sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFilterSuite.scala
-index 104b4e416cd..0897e4f3776 100644
+index 104b4e416cd..5ff142320cd 100644
 --- 
a/sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFilterSuite.scala
 +++ 
b/sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFilterSuite.scala
 @@ -38,6 +38,7 @@ import org.apache.parquet.schema.MessageType
@@ -2069,7 +2069,7 @@ index 104b4e416cd..0897e4f3776 100644
  
 -  test("filter pushdown - StringPredicate") {
 +  test("filter pushdown - StringPredicate",
-+    IgnoreCometNativeScan("cannot be pushed down")) {
++    
IgnoreCometNativeScan("https://github.com/apache/datafusion-comet/issues/3321";))
 {
      import testImplicits._
      // keep() should take effect on StartsWith/EndsWith/Contains
      Seq(


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to