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/07/18 13:47:51 UTC

[GitHub] [incubator-sedona] umartin commented on pull request #647: WIP: [SEDONA-132] Move some functions to a common module

umartin commented on PR #647:
URL: https://github.com/apache/incubator-sedona/pull/647#issuecomment-1187477821

   > 
   
   Doing it piece by piece is probably a good idea but it would be nice to have a clear end goal.
   
   Spatial partitioning and the kryoserializer are not intertwined with other apis and should be pretty easy to move.
   I don't think it's strictly nessesary to move format readers. Some ST-constructors depend on format readers. They could be refactored to call JTS and geojson libraries directly. Then the format readers could use the functions in common where it makes sense.


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