jiayuasu commented on PR #647: URL: https://github.com/apache/incubator-sedona/pull/647#issuecomment-1188246695
> > > > > > > > > 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. > > How about limiting this PR to the "Functions" defined in Flink? And then make separate issues for other things Flink already has (predicates, constructors), and then other things can be done as they are used/added to Flink I agree. Let's start with the functions in this PR. And then create other PRs for other functions. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
