Craig Taverner

That was a very interesting analysis. Neo4j Spatial was not originally written for high concurrency scenarios, but it is great if it can slowly be made more concurrency safe. I...

There are two pull-requests to upgrade geotools support (to versions 21 and 22 respectively) in the list of pull requests above. We hope to integrate one or both of these...

Using the Java API there is support for some additional things which might be more like what you are asking for. See https://github.com/neo4j-contrib/spatial/blob/master/src/main/java/org/neo4j/gis/spatial/SpatialTopologyUtils.java We could expose these through the spatial...

The workaround is to keep increasing the number of items until you do get hits. That could also be encoded into the algorithm (automatically increment estimated search window if too...

There is a procedure `spatial.setFeatureAttributes` which does this. It will set the list of known attributes at the layer level. However, those should also exist as properties on the nodes...

I like all suggestions. * Messy tests that output stuff outside target should be cleaned up to output to target * It would be good if travis CI ran unit...

Yes, ignore the test. I will find the file and unignore the test when I do.

Also, for me to merge your PR's, please make sure to agree to the CLA, as described at https://neo4j.com/developer/cla/

Since your fork is already implicitly another branch, I personally see no reason to make yet another branch. Just fix on your master and send the pull request. I think...

Sorry, clicked 'close and comment' instead of just 'comment' before. Slip of the fingers :-)