rmuir commented on PR #12162: URL: https://github.com/apache/lucene/pull/12162#issuecomment-1438428404
just look at the code to see what i mean: https://github.com/apache/lucene/blob/main/lucene/core/src/java/org/apache/lucene/document/LatLonPoint.java#L325-L341 The worst part is, the user who just wants to find points in their bounding box or simple polygon, doesn't think that they should be using `INTERSECTS` because the verb is senseless when thinking of something that is infinitely small. This disastrous api should be completely removed from LatLonPoint, too. Keep it for shapes if you want. -- 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-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
