You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sedona.apache.org by GitBox <gi...@apache.org> on 2022/09/26 08:56:57 UTC

[GitHub] [incubator-sedona] sekikn opened a new pull request, #694: [SEDONA-171] Add ST_SetPoint to Apache Sedona

sekikn opened a new pull request, #694:
URL: https://github.com/apache/incubator-sedona/pull/694

   
   ## Did you read the Contributor Guide?
   
   - Yes, I have read [Contributor Rules](https://sedona.apache.org/community/rule/) and [Contributor Development Guide](https://sedona.apache.org/community/develop/)
   
   ## Is this PR related to a JIRA ticket?
   
   - Yes, the URL of the assoicated JIRA ticket is https://issues.apache.org/jira/browse/SEDONA-171. The PR name follows the format `[SEDONA-XXX] my subject`.
   
   ## 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](https://github.com/apache/incubator-sedona/blob/master/pom.xml#L29) in since `vX.Y.Z` format.
   


-- 
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: dev-unsubscribe@sedona.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-sedona] jiayuasu commented on pull request #694: [SEDONA-171] Add ST_SetPoint to Apache Sedona

Posted by GitBox <gi...@apache.org>.
jiayuasu commented on PR #694:
URL: https://github.com/apache/incubator-sedona/pull/694#issuecomment-1258896133

   Looks good to me. Will wait until PR-693 gets merged (https://github.com/apache/incubator-sedona/pull/693). Then @sekikn may need to update this PR to add the type-safe Spark DataFrame API


-- 
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: dev-unsubscribe@sedona.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-sedona] jiayuasu merged pull request #694: [SEDONA-171] Add ST_SetPoint to Apache Sedona

Posted by GitBox <gi...@apache.org>.
jiayuasu merged PR #694:
URL: https://github.com/apache/incubator-sedona/pull/694


-- 
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: dev-unsubscribe@sedona.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-sedona] jiayuasu commented on pull request #694: [SEDONA-171] Add ST_SetPoint to Apache Sedona

Posted by GitBox <gi...@apache.org>.
jiayuasu commented on PR #694:
URL: https://github.com/apache/incubator-sedona/pull/694#issuecomment-1264706663

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


-- 
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: dev-unsubscribe@sedona.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org