Skip to content

Conversation

@uros-db
Copy link
Contributor

@uros-db uros-db commented Nov 2, 2025

What changes were proposed in this pull request?

Implement the st_srid function in Scala and PySpark API.

Why are the changes needed?

Expand API support for the ST_Srid expression.

Does this PR introduce any user-facing change?

Yes, the new function is now available in Scala and PySpark API.

How was this patch tested?

Added appropriate Scala function unit tests:

  • STFunctionsSuite

Added appropriate PySpark function unit tests:

  • test_functions

Was this patch authored or co-authored using generative AI tooling?

No.

@github-actions github-actions bot added the SQL label Nov 2, 2025
uros-db

This comment was marked as outdated.

Copy link
Contributor Author

@uros-db uros-db left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@uros-db uros-db changed the title [SPARK-54142][Geo][SQL] Implement the st_srid function in Scala [SPARK-54142][Geo][SQL][PYTHON] Implement the st_srid function in Scala and PySpark Nov 3, 2025
@uros-db
Copy link
Contributor Author

uros-db commented Nov 4, 2025

@zhengruifeng @HyukjinKwon @cloud-fan All checks good, ready to merge to master/4.1.

@HyukjinKwon
Copy link
Member

Merged to master and branch-4.1.

HyukjinKwon pushed a commit that referenced this pull request Nov 4, 2025
…la and PySpark

### What changes were proposed in this pull request?
Implement the `st_srid` function in Scala and PySpark API.

### Why are the changes needed?
Expand API support for the `ST_Srid` expression.

### Does this PR introduce _any_ user-facing change?
Yes, the new function is now available in Scala and PySpark API.

### How was this patch tested?
Added appropriate Scala function unit tests:
- `STFunctionsSuite`

Added appropriate PySpark function unit tests:
- `test_functions`

### Was this patch authored or co-authored using generative AI tooling?
No.

Closes #52841 from uros-db/geo-ST_Srid-scala.

Authored-by: Uros Bojanic <[email protected]>
Signed-off-by: Hyukjin Kwon <[email protected]>
(cherry picked from commit 8c76795)
Signed-off-by: Hyukjin Kwon <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants