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

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

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

   Can we add newPolygonQuery and newBoxQuery just like LatLonPoint already has?
   
   To me, the newGeometryQuery method is unusable and overcomplicated. No user should struggle with how to do a basic bounding box or point-in-polygon query with it (and it isn't obvious whatsoever).
   
   Let's give the user meaningful methods that match common use-cases.


-- 
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