This is an automated email from the ASF dual-hosted git repository.
cloud-fan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new c78c0c86c77d [SPARK-57434][SQL][TEST] Move JavaGeographyTypeSuite and
JavaGeometryTypeSuite to the Java test source root
c78c0c86c77d is described below
commit c78c0c86c77d927d1e704a7065a9affae37d6734
Author: Wenchen Fan <[email protected]>
AuthorDate: Sun Jun 14 07:48:56 2026 -0700
[SPARK-57434][SQL][TEST] Move JavaGeographyTypeSuite and
JavaGeometryTypeSuite to the Java test source root
### What changes were proposed in this pull request?
Move `JavaGeographyTypeSuite` and `JavaGeometryTypeSuite` from
`sql/core/src/test/scala/org/apache/spark/sql/types/` to
`sql/core/src/test/java/org/apache/spark/sql/types/`.
These are Java (JUnit) test suites that were placed under the Scala test
source
root when they were introduced in #52623 ([SPARK-53920]). This relocates
them to
the standard Java test source root. The package
(`org.apache.spark.sql.types`)
is unchanged and there is no logic change.
### Why are the changes needed?
Java test sources belong under `src/test/java`, not `src/test/scala`.
Keeping
them in the Scala source tree is inconsistent with the standard Maven/SBT
project layout.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Existing tests. This is a pure file relocation with no logic change.
### Was this patch authored or co-authored using generative AI tooling?
Yes, prepared with assistance from an AI coding agent.
Closes #56492 from cloud-fan/move-java-geo-typesuites-to-java-root.
Authored-by: Wenchen Fan <[email protected]>
Signed-off-by: Wenchen Fan <[email protected]>
---
.../org/apache/spark/sql/types/JavaGeographyTypeSuite.java | 0
.../org/apache/spark/sql/types/JavaGeometryTypeSuite.java | 4 ++--
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git
a/sql/core/src/test/scala/org/apache/spark/sql/types/JavaGeographyTypeSuite.java
b/sql/core/src/test/java/org/apache/spark/sql/types/JavaGeographyTypeSuite.java
similarity index 100%
rename from
sql/core/src/test/scala/org/apache/spark/sql/types/JavaGeographyTypeSuite.java
rename to
sql/core/src/test/java/org/apache/spark/sql/types/JavaGeographyTypeSuite.java
diff --git
a/sql/core/src/test/scala/org/apache/spark/sql/types/JavaGeometryTypeSuite.java
b/sql/core/src/test/java/org/apache/spark/sql/types/JavaGeometryTypeSuite.java
similarity index 97%
rename from
sql/core/src/test/scala/org/apache/spark/sql/types/JavaGeometryTypeSuite.java
rename to
sql/core/src/test/java/org/apache/spark/sql/types/JavaGeometryTypeSuite.java
index 867957884d23..9a19367a25c2 100644
---
a/sql/core/src/test/scala/org/apache/spark/sql/types/JavaGeometryTypeSuite.java
+++
b/sql/core/src/test/java/org/apache/spark/sql/types/JavaGeometryTypeSuite.java
@@ -64,8 +64,8 @@ public class JavaGeometryTypeSuite {
public void geometryTypeWithSpecifiedValidCrsTest() {
// Valid CRS values for GEOMETRY (including Spark overrides)
Stream.of(
- "SRID:0", "EPSG:3857", "OGC:CRS84", "EPSG:4326", "OGC:CRS27",
"EPSG:4267", "OGC:CRS83", "EPSG:4269",
- "EPSG:2000", "ESRI:102100")
+ "SRID:0", "EPSG:3857", "OGC:CRS84", "EPSG:4326", "OGC:CRS27",
"EPSG:4267",
+ "OGC:CRS83", "EPSG:4269", "EPSG:2000", "ESRI:102100")
.forEach(crs -> {
Integer srid = CartesianSpatialReferenceSystemMapper.getSrid(crs);
DataType geometryType = DataTypes.createGeometryType(crs);
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
