[SEDONA-659] Upgrade JTS version to 1.20.0 - Githubissues

apache / sedona

A cluster computing framework for processing large-scale geospatial data

https://sedona.apache.org/

Apache License 2.0

1.96k stars 692 forks source link

[SEDONA-659] Upgrade JTS version to 1.20.0 #1655

Closed Kontinuation closed 3 weeks ago

Kontinuation commented 4 weeks ago

Did you read the Contributor Guide?

Yes, I have read the Contributor Rules and Contributor Development Guide

Is this PR related to a JIRA ticket?

Yes, the URL of the associated JIRA ticket is https://issues.apache.org/jira/browse/SEDONA-659. The PR name follows the format [SEDONA-XXX] my subject.

What changes were proposed in this PR?

Upgrade JTS from 1.19.0 to 1.20.0.

Most of the changes are made to the tests due to different computation results obtained from 1.20.0. The following functions are affected from the observations of broken tests:

ST_MaximumIncribedCircle
ST_VoronoiPolygons
ST_TriangulatePolygon
ST_Buffer
ST_ConcaveHull

ST_MaximumIncribedCircle has significant changes, while other functions only introduce slight coordinate value changes.

How was this patch tested?

Passing tests.

Did this PR include necessary documentation updates?

No, this PR does not affect any public API so no need to change the documentation.

Kontinuation commented 4 weeks ago

PR for running snowflake tests: https://github.com/wherobots/sedona-snowflake-tester/pull/38