jiayuasu opened a new pull request, #3206: URL: https://github.com/apache/sedona/pull/3206
## Did you read the Contributor Guide? - Yes, I have read the [Contributor Rules](https://sedona.apache.org/latest/community/rule/) and [Contributor Development Guide](https://sedona.apache.org/latest/community/develop/). ## Is this PR related to a ticket? - Yes, and the PR name follows the format `[GH-XXX] my subject`. Closes #3201. ## What changes were proposed in this PR? - Return an empty Geography unchanged from `ST_Envelope`, preserving its subtype, SRID, and object identity. - Detect WKB `POINT EMPTY` directly before constructing an `S2PointRegion` from its NaN ordinates. - Return the input when another empty Geography region produces an empty S2 rectangle. - Apply the same behavior for both values of `splitAtAntiMeridian`; SQL `NULL` behavior remains unchanged. This matches Sedona's existing Geometry `ST_Envelope` behavior for typed empty values. Empty LineString WKB serialization is an independent defect covered by #3200. ## How was this patch tested? - Common Geography `FunctionTest`: 78 tests, 0 failures. - Spark Geography DataFrame API suite: 19 tests, 0 failures. - Flink Geography function suite: 18 tests, 0 failures. - Common coverage includes empty Point, LineString, Polygon, and GeometryCollection values with both antimeridian settings. - Spark and Flink integration coverage verifies `POINT EMPTY`, SRID preservation, and both settings. - Spotless and commit hooks passed. ## Did this PR include necessary documentation updates? - No. This fixes an exception for an existing API and aligns it with the established empty Geometry behavior. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
