[GitHub] [lucene] iverase commented on pull request #12162: Add LatLonField class to index both LatLonPoint and LatLonDocValues

["iverase (via GitHub)" <gi...@apache.org>]

iverase commented on PR #12162:
URL: https://github.com/apache/lucene/pull/12162#issuecomment-1437092325

   Sorry but I totally disagree, this is the most typical API for a geo application so if someone else struggle with it then it should not be developing a geo application. 
   Anyway I don't care about those methods so I have added them if it makes the PR moves forward.
   
   -1 for removing newGeometryQuery(), this allows to use LatLonShape and LatLonPoint under a common API. This is the only method I care for, so sorry no removing it please.
   


-- 
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: issues-unsubscribe@lucene.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org