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/03/20 20:37:40 UTC

[GitHub] [incubator-sedona] netanel246 opened a new pull request #593: Add ST_GeomFromGeoJSON to flink API

netanel246 opened a new pull request #593:
URL: https://github.com/apache/incubator-sedona/pull/593


   ## Is this PR related to a proposed Issue?
   [SEDONA-93](https://issues.apache.org/jira/browse/SEDONA-93)
   ## What changes were proposed in this PR?
   Added ST_GeomFromGeoJSON to flink API
   ## How was this patch tested?
   Added test
   ## Did this PR include necessary documentation updates?
   Yep


-- 
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 #593: [SEDONA-93] Add ST_GeomFromGeoJSON to flink API

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


   @netanel246 Thanks for your contribution. This looks good. 
   
   Some of the functions in Sedona Spark SQL are from JTS and some are standalone (without Spark dependency). So I think we can just call those functions in Sedona Flink SQL. No need to create a separate Maven module for it at this moment but maybe in the future.


-- 
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 #593: [SEDONA-93] Add ST_GeomFromGeoJSON to flink API

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


   


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