Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SEDONA-693] Add ST_Perimeter2D #47

Closed
wants to merge 8 commits into from

Conversation

furqaankhan
Copy link
Collaborator

Did you read the Contributor 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-XXX. The PR name follows the format [SEDONA-XXX] my subject.

  • No:

    • this is a documentation update. The PR name follows the format [DOCS] my subject
    • this is a CI update. The PR name follows the format [CI] my subject

What changes were proposed in this PR?

How was this patch tested?

Did this PR include necessary documentation updates?

  • Yes, I am adding a new API. I am using the current SNAPSHOT version number in vX.Y.Z format.
  • Yes, I have updated the documentation.
  • No, this PR does not affect any public API so no need to change the documentation.

james-willis and others added 6 commits December 19, 2024 16:47
* add instructions for testing different spark/scala versions in Intellij

* recommend JDK version for project rather than ide setting

---------

Co-authored-by: jameswillis <[email protected]>
* Geostats SQL

* add missing import statement

---------

Co-authored-by: jameswillis <[email protected]>
* [SEDONA-688] Verify KNN parameter K must be equal or larger than 1

* [SEDONA-690] Optimize query side broadcast knn join

* fix isGeography parameter
@furqaankhan furqaankhan requested a review from jiayuasu as a code owner January 7, 2025 05:12
@furqaankhan furqaankhan changed the title ST_Perimeter2D [SEDONA-693] Add ST_Perimeter2D Jan 7, 2025
@furqaankhan furqaankhan closed this Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants