lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (JIRA)" <>
Subject [jira] [Commented] (LUCENE-7160) LatLonPoint quantization should use same rounding mode as GeoPointField
Date Fri, 01 Apr 2016 12:37:25 GMT


ASF subversion and git services commented on LUCENE-7160:

Commit d9d88b8941108bf9d30c1e47ab881e94e6faabf2 in lucene-solr's branch refs/heads/branch_6x
from [~rcmuir]
[;h=d9d88b8 ]

LUCENE-7160: LatLonPoint quantization should use the same rounding mode as GeoPointField

> LatLonPoint quantization should use same rounding mode as GeoPointField
> -----------------------------------------------------------------------
>                 Key: LUCENE-7160
>                 URL:
>             Project: Lucene - Core
>          Issue Type: Bug
>            Reporter: Robert Muir
>         Attachments: LUCENE-7160.patch
> Both of these fields quantize doubles in a lossy way, but GeoPoint rounds towards negative
infinity (because all values get shifted to positive space first), whereas LatLonPoint currently
rounds towards zero.
> At the same time they share the same basic logic for e.g. tree traversal and same basic
tests: so it creates confusion that they round differently.
> Especially since e.g. LatLonPoint does some operations in binary/integer space (some
of these optimizations such as its box query are currently buggy for edge cases, and tests
are lenient as TODO, another issue, may or may not impact GeoPoint, it at least tries harder).
> I think LatLonPoint should round down to be more like GeoPoint here? By the way, doing
this also gives more consistency, it reduces the difference between the two implementations
in e.g. openstreetmaps searching (distance query benchmark now differs by only 4 hits...)
> This quantization makes things complicated to reason about. I understand the advantages
it brings but I think along with that comes the responsibility of us handling edge cases better.
I don't think we do well at this today...

This message was sent by Atlassian JIRA

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message