jiayuasu opened a new pull request, #3272: URL: https://github.com/apache/sedona/pull/3272
## 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`. Follow-up to #3250 (GH-2800). ## What changes were proposed in this PR? Running the Snowflake tester against a real Snowflake environment surfaced two failures introduced by #3250: - `TestFunctions.test_ST_Translate`: #3250 fixed the WKT writer to emit `LINESTRING Z EMPTY` instead of JTS's malformed `LINESTRING ZEMPTY`, but this test still expected the old output. Updated the expectation. - `TestFunctionsV2.test_ST_SharedPaths`: the result contained `MULTILINESTRING EMPTY`, and V2 UDFs return Snowflake-native GEOMETRY via GeoJSON, which Snowflake rejects ("Invalid GeoJSON. 'coordinates' must not be empty."). Switched to inputs that share paths in both directions so no collection element is empty; the V1 test keeps the empty-element coverage since it round-trips through Sedona's own serialization. ## How was this patch tested? Verified against a real Snowflake environment: the full `snowflake-tester` suite passes with this change (wherobots/sedona-cloud-vendor-tester#73, run https://github.com/wherobots/sedona-cloud-vendor-tester/actions/runs/32258418019). ## Did this PR include necessary documentation updates? - No, this PR does not affect any public API so no need to change the documentation. -- 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]
