[GitHub] [incubator-sedona] jiayuasu edited a comment on pull request #482: [SEDONA-1] Move to jts 1.16.1

[GitBox <gi...@apache.org>]

jiayuasu edited a comment on pull request #482:
URL: https://github.com/apache/incubator-sedona/pull/482#issuecomment-689441598


   @nirnir93 Ok, if so, I think this will significantly decrease our performance. Our existing serializer outperforms Kryo in terms of speed and memory footprint. It can serialize both geometries and spatial indices. This is one of the reason why Sedona beats competitors in many benchmarks using large-scale datasets.
   
   If there is no other way to bring back the serializer, in the worst case, I will create a fork on LocationTech JTS and periodically and manually pull the latest JTS functions to JTSplus. In this case, no code change is needed for Sedona. But it requires some coding work in the long run and JTS and JTSplus will conflict if they co-exist in one project.


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

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