jiayuasu commented on pull request #488:
URL: https://github.com/apache/incubator-sedona/pull/488#issuecomment-727784872


   Note that, this PR depends on a JTS 1.16.2 fork: 
https://github.com/jiayuasu/jts
   
   1. Although we won't be able to use JTS 1.17 official release (due to the 
issues among JTStoGeoJSON dependency, GeoTools dependency and JTS) even if both 
two JTS PRs get accepted, this at least can help us to upgrade JTS 1.17 in the 
future without any API changes.
   
   2. If the first JTS PR https://github.com/locationtech/jts/pull/633 is 
declined by JTS committers, we have to remove SpatialJoinQuery <Geom, 
HashSet<Geom>> function or use a so-called custom HashStrategy. Because Java 
built-in HashSet has to Geom.equals(object) to compute the HashCode.
   
   @Imbruced Could you please fix the UserData errors in Python accordingly? I 
have invited you to be the collaborator on my Sedona fork.


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to