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-171] Add ST_SetPoint to Apache Sedona #694

Merged
merged 5 commits into from
Oct 9, 2022

Conversation

sekikn
Copy link
Contributor

@sekikn sekikn commented Sep 26, 2022

Did you read the Contributor Guide?

Is this PR related to a JIRA ticket?

What changes were proposed in this PR?

  • This PR adds the ST_SetPoint function to the Spark and Flink API.

How was this patch tested?

  • Ran mvn clean install -Dgeotools locally
  • Ran mkdocs serve and checked the generated documents

Did this PR include necessary documentation updates?

  • Yes, I am adding a new API. I am using the 1.3.0 in since vX.Y.Z format.

@jiayuasu
Copy link
Member

Looks good to me. Will wait until PR-693 gets merged (#693). Then @sekikn may need to update this PR to add the type-safe Spark DataFrame API

@jiayuasu
Copy link
Member

jiayuasu commented Oct 2, 2022

@sekikn PR #693 is merged, would you please add the DataFrame style API (in both Scala and Python) to your SQL function? Thank you!

@jiayuasu jiayuasu merged commit 206936a into apache:master Oct 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants