David Smiley

I believe @romseygeek worked on a HTTP based Luke-like thing and invested a lot of time into it.

I think the point here isn't the merits of one version over another, it's about the pom stating the version explicitly: https://github.com/locationtech/spatial4j/blob/master/pom.xml#L172 IMO every project ought to do this explicitly...

For posterity: Java 1.8 is the version JTS uses as of v 1.17. I see it in the release notes too (good). It seems to have come about accidentally but...

I'm not sure we _need_ to indicate there are unexpected XML elements. I'm reminded of the "Robustness Principle" -- https://en.wikipedia.org/wiki/Robustness_principle which is "Be conservative in what you send, be liberal...

I just want to say "thank you" for an unexpectedly thorough review by @mkhludnev on a PR that many of us saw and passed on (I did).

I get interaction with maintainers on other issues promptly. Granted they were mostly resolved with changes that me as a user could do and didn't lead to code changes.

Can you share the NPE stack trace please? At least Spatial4J has multiple options for GeoJSON reading. I recall JTS has one as well, and off the top of my...

Sorry this may be out of scope of this repo/issues here but related to Zooma... is the code behind that available? I was poking around the site but I didn't...

Thanks. Wow there's a lot of code there; it will take some time to explore to find the parts similar to the SolrTextTagger -- which is scanning text to look...

I propose we make the loss of the reconnect boolean its own PR, plus the methods that shielded use of ZkCmdExecutor. That will make this effort here more independently reviewable...