jiayuasu opened a new pull request, #3207:
URL: https://github.com/apache/sedona/pull/3207

   ## 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 
#3203.
   
   ## What changes were proposed in this PR?
   
   - Render default `WKBGeography.toString()` output from the authoritative 
stored WKB/JTS representation instead of the derived S2 representation.
   - Build default `toEWKT()` from the same structural text and preserve 
positive wrapper SRIDs.
   - Preserve ordinary coordinates such as `POINT (-122.4194 37.7749)` without 
the default S2 fixed-precision truncation to `POINT (-122.4 37.8)`.
   - Preserve repeated LineString vertices in default object text.
   - Keep the explicit `PrecisionModel` overloads on the S2 writer so callers 
can still request the previous precision-controlled representation.
   
   Spark DataFrame display uses the object's default text representation, so 
this aligns `df.show()` with WKB-backed structural accessors such as 
`ST_AsText`.
   
   This is an intentional output change. S2-derived producers can already store 
binary floating-point artifacts in WKB, and default object text will now expose 
that structural value just as `ST_AsText` does. The explicit precision overload 
remains available for normalized display. #3195 separately avoids the initial 
S2 round trip for WKT-created Geography values.
   
   ## How was this patch tested?
   
   - Full common module suite: 1,182 tests, 0 failures.
   - Focused `WKBGeographyTest` and Geography constructor suites: 44 tests, 0 
failures.
   - Added coverage for exact ordinary point coordinates, EWKT SRID 
preservation, repeated LineString vertices, and the unchanged explicit 
fixed-precision path.
   - Spotless and commit hooks passed.
   
   ## Did this PR include necessary documentation updates?
   
   - No. This changes the Java object display representation to agree with the 
existing WKB-backed SQL text representation; SQL API signatures are unchanged.
   


-- 
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]

Reply via email to