Doesn't the built in Lucene implementation in Geode have geospatial indexing already?
-- Mike Stolz Principal Engineer, GemFire Product Lead Mobile: +1-631-835-4771 Download the GemFire book here. <https://content.pivotal.io/ebooks/scaling-data-services-with-pivotal-gemfire> On Wed, Sep 5, 2018 at 1:23 PM, galen-pivotal <[email protected]> wrote: > *@galen-pivotal* requested changes on this pull request. > > I'm still concerned about the geospatial code so closely matching Redis > code -- some of the comments looked like they were directly copied. Even > though Redis is open source, licensing is hard, because code that is a > derivative work of Redis would have to be BSD-licensed, but code that is > included in Geode must be Apache-licensed. Perhaps we could use an existing > geospatial library instead? > > I found these two on a quick net search: > https://github.com/kungfoo/geohash-java > https://github.com/davidmoten/geo > > — > You are receiving this because you are subscribed to this thread. > Reply to this email directly, view it on GitHub > <https://github.com/apache/geode/pull/2210#pullrequestreview-152604282>, > or mute the thread > <https://github.com/notifications/unsubscribe-auth/ASblIAzFa999XT-xNk_HCdbhxdveU3eBks5uYAiEgaJpZM4Vispk> > . > [ Full content available at: https://github.com/apache/geode/pull/2210 ] This message was relayed via gitbox.apache.org for [email protected]
