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

agrove pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/datafusion-comet.git


The following commit(s) were added to refs/heads/main by this push:
     new e02b6cd42 Enable tests in RemoveRedundantProjectsSuite.scala related 
to #242 (#1838)
e02b6cd42 is described below

commit e02b6cd42d98730b267e5c25b83c23f756e809ad
Author: Rishab Joshi <8187657+rish...@users.noreply.github.com>
AuthorDate: Wed Jun 4 12:11:00 2025 -0700

    Enable tests in RemoveRedundantProjectsSuite.scala related to #242 (#1838)
---
 dev/diffs/3.5.5.diff | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/dev/diffs/3.5.5.diff b/dev/diffs/3.5.5.diff
index ee17eab75..310aa6881 100644
--- a/dev/diffs/3.5.5.diff
+++ b/dev/diffs/3.5.5.diff
@@ -1189,7 +1189,7 @@ index 9e9d717db3b..c1a7caf56e0 100644
  package org.apache.spark.sql.execution
  
 -import org.apache.spark.sql.{DataFrame, QueryTest, Row}
-+import org.apache.spark.sql.{DataFrame, IgnoreComet, QueryTest, Row}
++import org.apache.spark.sql.{DataFrame, QueryTest, Row}
 +import org.apache.spark.sql.comet.CometProjectExec
  import org.apache.spark.sql.connector.SimpleWritableDataSource
  import org.apache.spark.sql.execution.adaptive.{AdaptiveSparkPlanHelper, 
DisableAdaptiveExecutionSuite, EnableAdaptiveExecutionSuite}
@@ -1206,13 +1206,12 @@ index 9e9d717db3b..c1a7caf56e0 100644
        assert(actual == expected)
      }
    }
-@@ -112,7 +116,8 @@ abstract class RemoveRedundantProjectsSuiteBase
+@@ -112,7 +116,7 @@ abstract class RemoveRedundantProjectsSuiteBase
      assertProjectExec(query, 1, 3)
    }
  
 -  test("join with ordering requirement") {
-+  test("join with ordering requirement",
-+    IgnoreComet("TODO: Support SubqueryBroadcastExec in Comet: #242")) {
++  test("join with ordering requirement") {
      val query = "select * from (select key, a, c, b from testView) as t1 join 
" +
        "(select key, a, b, c from testView) as t2 on t1.key = t2.key where 
t2.a > 50"
      assertProjectExec(query, 2, 2)


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@datafusion.apache.org
For additional commands, e-mail: commits-h...@datafusion.apache.org

Reply via email to