diff --git a/pom.xml b/pom.xml
index c7e417dd1b..f5f9bf26f1 100644
--- a/pom.xml
+++ b/pom.xml
@@ -86,7 +86,8 @@ under the License.
-Djdk.reflect.useDirectMethodHandle=false
-ea -Xmx4g -Xss4m ${extraJavaTestArgs}
- spark-3.3-plus
+ spark-3.3-plus
+ spark-3.4
@@ -496,7 +497,8 @@ under the License.
3.21.12.0
- not-needed-yet
+ not-needed-yet
+ not-needed-yet
@@ -507,7 +509,8 @@ under the License.
3.3.23.31.12.0
- spark-3.3-plus
+ spark-3.3-plus
+ not-needed-yet
@@ -517,7 +520,8 @@ under the License.
2.12.173.41.13.1
- spark-3.3-plus
+ spark-3.3-plus
+ spark-3.4
diff --git a/spark/pom.xml b/spark/pom.xml
index 31d80bbe69..e0be7c6562 100644
--- a/spark/pom.xml
+++ b/spark/pom.xml
@@ -245,7 +245,8 @@ under the License.
-
+
+
diff --git a/spark/src/test/scala/org/apache/comet/exec/CometExecSuite.scala b/spark/src/test/scala/org/apache/comet/exec/CometExecSuite.scala
index 170ee5e4b4..a8b05cc984 100644
--- a/spark/src/test/scala/org/apache/comet/exec/CometExecSuite.scala
+++ b/spark/src/test/scala/org/apache/comet/exec/CometExecSuite.scala
@@ -60,15 +60,6 @@ class CometExecSuite extends CometTestBase {
}
}
- test("offset") {
- assume(isSpark34Plus, "Dataset.offset is not available before Spark 3.4")
- withSQLConf(CometConf.COMET_COLUMNAR_SHUFFLE_ENABLED.key -> "true") {
- checkSparkAnswer(testData.offset(90))
- checkSparkAnswer(arrayData.toDF().offset(99))
- checkSparkAnswer(mapData.toDF().offset(99))
- }
- }
-
test("try_sum should return null if overflow happens before merging") {
assume(isSpark33Plus, "try_sum is available in Spark 3.3+")
val longDf = Seq(Long.MaxValue, Long.MaxValue, 2).toDF("v")
diff --git a/spark/src/test/spark-3.4/org/apache/comet/exec/CometExec3_4Suite.scala b/spark/src/test/spark-3.4/org/apache/comet/exec/CometExec3_4Suite.scala
new file mode 100644
index 0000000000..ae43861a4b
--- /dev/null
+++ b/spark/src/test/spark-3.4/org/apache/comet/exec/CometExec3_4Suite.scala
@@ -0,0 +1,52 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+package org.apache.comet.exec
+
+import org.scalactic.source.Position
+import org.scalatest.Tag
+
+import org.apache.spark.sql.CometTestBase
+
+import org.apache.comet.CometConf
+
+/**
+ * This test suite contains tests for only Spark 3.4.
+ */
+class CometExec3_4Suite extends CometTestBase {
+ import testImplicits._
+
+ override protected def test(testName: String, testTags: Tag*)(testFun: => Any)(implicit
+ pos: Position): Unit = {
+ super.test(testName, testTags: _*) {
+ withSQLConf(CometConf.COMET_EXEC_SHUFFLE_ENABLED.key -> "true") {
+ testFun
+ }
+ }
+ }
+
+ // Dataset.offset API is not available before Spark 3.4
+ test("offset") {
+ withSQLConf(CometConf.COMET_COLUMNAR_SHUFFLE_ENABLED.key -> "true") {
+ checkSparkAnswer(testData.offset(90))
+ checkSparkAnswer(arrayData.toDF().offset(99))
+ checkSparkAnswer(mapData.toDF().offset(99))
+ }
+ }
+}