This is an automated email from the ASF dual-hosted git repository.
dongjoon-hyun pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/spark-connect-swift.git
The following commit(s) were added to refs/heads/main by this push:
new e7307b6 [SPARK-57347] Remove `spark.sql.geospatial.enabled=true` from
`integration-test-mac-spark42`
e7307b6 is described below
commit e7307b6b113e155d60da5a8c907404c430f71f92
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Tue Jun 9 09:46:36 2026 -0700
[SPARK-57347] Remove `spark.sql.geospatial.enabled=true` from
`integration-test-mac-spark42`
### What changes were proposed in this pull request?
This PR removes the `-c spark.sql.geospatial.enabled=true` configuration
from the `start-connect-server.sh` invocation in the
`integration-test-mac-spark42` job of `.github/workflows/build_and_test.yml`.
```diff
- ./start-connect-server.sh -c spark.sql.geospatial.enabled=true -c
spark.sql.timeType.enabled=true
+ ./start-connect-server.sh -c spark.sql.timeType.enabled=true
```
### Why are the changes needed?
From Apache Spark 4.2.0 RC1, geospatial support (`GEOMETRY`/`GEOGRAPHY`
types) is enabled by default, so explicitly passing
`spark.sql.geospatial.enabled=true` is redundant.
- https://github.com/apache/spark/pull/55734
### Does this PR introduce _any_ user-facing change?
No. This is a CI-only change.
### How was this patch tested?
Pass the CI, specifically the `integration-test-mac-spark42` job which runs
the full test suite (including the `geography.sql` and `geometry.sql` golden
tests) against a Spark 4.2.0 RC1 Connect server.
### Was this patch authored or co-authored using generative AI tooling?
Generated-by: Claude Code (Claude Opus 4.8)
Closes #417 from dongjoon-hyun/SPARK-57347.
Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
---
.github/workflows/build_and_test.yml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/.github/workflows/build_and_test.yml
b/.github/workflows/build_and_test.yml
index cf614cc..65879f5 100644
--- a/.github/workflows/build_and_test.yml
+++ b/.github/workflows/build_and_test.yml
@@ -159,7 +159,7 @@ jobs:
tar xvfz spark-4.2.0-bin-hadoop3.tgz && rm spark-4.2.0-bin-hadoop3.tgz
mv spark-4.2.0-bin-hadoop3/ /tmp/spark
cd /tmp/spark/sbin
- ./start-connect-server.sh -c spark.sql.geospatial.enabled=true -c
spark.sql.timeType.enabled=true
+ ./start-connect-server.sh -c spark.sql.timeType.enabled=true
cd -
swift test --no-parallel -c release
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
