This is an automated email from the ASF dual-hosted git repository.
maxgekk pushed a commit to branch branch-4.0
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/branch-4.0 by this push:
new c556e24ae4fc [SPARK-51502][SQL][TESTS] Move collations test to
collations package
c556e24ae4fc is described below
commit c556e24ae4fc7698f6b2d10305ccc81023a95fad
Author: Stefan Kandic <stefan.kan...@databricks.com>
AuthorDate: Fri Mar 14 19:03:18 2025 +0300
[SPARK-51502][SQL][TESTS] Move collations test to collations package
### What changes were proposed in this pull request?
Move collations test into collations package where most collation test
suites already are located.
### Why are the changes needed?
For consistency purposes, also not to bloat the sql folder with too many
suites.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Existing tests should suffice.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #50268 from stefankandic/moveCollTests.
Authored-by: Stefan Kandic <stefan.kan...@databricks.com>
Signed-off-by: Max Gekk <max.g...@gmail.com>
(cherry picked from commit 63f629686494c11976f4ee8c20cb8c72fa47459d)
Signed-off-by: Max Gekk <max.g...@gmail.com>
---
.../spark/sql/{ => collation}/CollationExpressionWalkerSuite.scala | 3 ++-
.../org/apache/spark/sql/{ => collation}/CollationSQLRegexpSuite.scala | 3 ++-
.../scala/org/apache/spark/sql/{ => collation}/CollationSuite.scala | 3 ++-
3 files changed, 6 insertions(+), 3 deletions(-)
diff --git
a/sql/core/src/test/scala/org/apache/spark/sql/CollationExpressionWalkerSuite.scala
b/sql/core/src/test/scala/org/apache/spark/sql/collation/CollationExpressionWalkerSuite.scala
similarity index 99%
rename from
sql/core/src/test/scala/org/apache/spark/sql/CollationExpressionWalkerSuite.scala
rename to
sql/core/src/test/scala/org/apache/spark/sql/collation/CollationExpressionWalkerSuite.scala
index 0d8a61d1932e..ba1be1b03ef8 100644
---
a/sql/core/src/test/scala/org/apache/spark/sql/CollationExpressionWalkerSuite.scala
+++
b/sql/core/src/test/scala/org/apache/spark/sql/collation/CollationExpressionWalkerSuite.scala
@@ -15,7 +15,7 @@
* limitations under the License.
*/
-package org.apache.spark.sql
+package org.apache.spark.sql.collation
import java.sql.Timestamp
@@ -23,6 +23,7 @@ import org.apache.spark.SparkFunSuite
import org.apache.spark.sql.catalyst.analysis.ExpressionBuilder
import org.apache.spark.sql.catalyst.expressions._
import org.apache.spark.sql.catalyst.expressions.variant.ParseJson
+import org.apache.spark.sql.classic
import org.apache.spark.sql.internal.SQLConf
import org.apache.spark.sql.internal.types._
import org.apache.spark.sql.test.SharedSparkSession
diff --git
a/sql/core/src/test/scala/org/apache/spark/sql/CollationSQLRegexpSuite.scala
b/sql/core/src/test/scala/org/apache/spark/sql/collation/CollationSQLRegexpSuite.scala
similarity index 99%
rename from
sql/core/src/test/scala/org/apache/spark/sql/CollationSQLRegexpSuite.scala
rename to
sql/core/src/test/scala/org/apache/spark/sql/collation/CollationSQLRegexpSuite.scala
index 87c399f895ad..c05ee529b529 100644
--- a/sql/core/src/test/scala/org/apache/spark/sql/CollationSQLRegexpSuite.scala
+++
b/sql/core/src/test/scala/org/apache/spark/sql/collation/CollationSQLRegexpSuite.scala
@@ -15,8 +15,9 @@
* limitations under the License.
*/
-package org.apache.spark.sql
+package org.apache.spark.sql.collation
+import org.apache.spark.sql.{AnalysisException, QueryTest, Row}
import org.apache.spark.sql.catalyst.expressions._
import org.apache.spark.sql.catalyst.plans.logical.Project
import org.apache.spark.sql.test.SharedSparkSession
diff --git a/sql/core/src/test/scala/org/apache/spark/sql/CollationSuite.scala
b/sql/core/src/test/scala/org/apache/spark/sql/collation/CollationSuite.scala
similarity index 99%
rename from sql/core/src/test/scala/org/apache/spark/sql/CollationSuite.scala
rename to
sql/core/src/test/scala/org/apache/spark/sql/collation/CollationSuite.scala
index f60b7afe816d..11e9547dfc5e 100644
--- a/sql/core/src/test/scala/org/apache/spark/sql/CollationSuite.scala
+++
b/sql/core/src/test/scala/org/apache/spark/sql/collation/CollationSuite.scala
@@ -15,11 +15,12 @@
* limitations under the License.
*/
-package org.apache.spark.sql
+package org.apache.spark.sql.collation
import scala.jdk.CollectionConverters.MapHasAsJava
import org.apache.spark.SparkException
+import org.apache.spark.sql.{AnalysisException, Row}
import org.apache.spark.sql.catalyst.ExtendedAnalysisException
import org.apache.spark.sql.catalyst.expressions._
import org.apache.spark.sql.catalyst.util.CollationFactory
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org
