Skip to content

Conversation

@uros-db
Copy link
Contributor

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

What changes were proposed in this pull request?

Introduce Arrow serialization/deserialization for Geography and Geometry.

Why are the changes needed?

Enable geospatial result set serialization in Arrow format for Spark Connect and Thrift Server.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Added unit tests:

  • GeographyConnectDataFrameSuite
  • GeometryConnectDataFrameSuite
  • ArrowEncoderSuite

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

No.

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.

@cloud-fan
Copy link
Contributor

thanks, merging to master/4.1!

@cloud-fan cloud-fan closed this in f919124 Nov 7, 2025
cloud-fan pushed a commit that referenced this pull request Nov 7, 2025
…and Geometry types

### What changes were proposed in this pull request?
Introduce Arrow serialization/deserialization for `Geography` and `Geometry`.

### Why are the changes needed?
Enable geospatial result set serialization in Arrow format for Spark Connect and Thrift Server.

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?
Added unit tests:
- `GeographyConnectDataFrameSuite`
- `GeometryConnectDataFrameSuite`
- `ArrowEncoderSuite`

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

Closes #52930 from uros-db/geo-arrow.

Authored-by: Uros Bojanic <[email protected]>
Signed-off-by: Wenchen Fan <[email protected]>
(cherry picked from commit f919124)
Signed-off-by: Wenchen Fan <[email protected]>
zifeif2 pushed a commit to zifeif2/spark that referenced this pull request Nov 22, 2025
…and Geometry types

### What changes were proposed in this pull request?
Introduce Arrow serialization/deserialization for `Geography` and `Geometry`.

### Why are the changes needed?
Enable geospatial result set serialization in Arrow format for Spark Connect and Thrift Server.

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?
Added unit tests:
- `GeographyConnectDataFrameSuite`
- `GeometryConnectDataFrameSuite`
- `ArrowEncoderSuite`

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

Closes apache#52930 from uros-db/geo-arrow.

Authored-by: Uros Bojanic <[email protected]>
Signed-off-by: Wenchen Fan <[email protected]>
huangxiaopingRD pushed a commit to huangxiaopingRD/spark that referenced this pull request Nov 25, 2025
…and Geometry types

### What changes were proposed in this pull request?
Introduce Arrow serialization/deserialization for `Geography` and `Geometry`.

### Why are the changes needed?
Enable geospatial result set serialization in Arrow format for Spark Connect and Thrift Server.

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?
Added unit tests:
- `GeographyConnectDataFrameSuite`
- `GeometryConnectDataFrameSuite`
- `ArrowEncoderSuite`

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

Closes apache#52930 from uros-db/geo-arrow.

Authored-by: Uros Bojanic <[email protected]>
Signed-off-by: Wenchen Fan <[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.

2 participants