Description
LatLonPoint methods are calling *precise() methods such as rectCrossesPolyPrecise(). GeoPointMethods are calling *approx() methods such as rectCrossesPolyApprox().
I don't understand why we have two versions of these methods, why one is approximate, why we would ever use that, etc.
These geo methods are extremely expensive: we can't afford two copies. For example, in the case of these polygon methods, they are still missing direct tests. We need to remove one copy of these. I propose to remove the approx ones.
Attachments
Issue Links
- is part of
-
LUCENE-7153 give GeoPoint and LatLonPoint full polygon support
-
- Closed
-