[ 
https://issues.apache.org/jira/browse/LUCENE-6930?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15057865#comment-15057865
 ] 

Michael McCandless commented on LUCENE-6930:
--------------------------------------------

+1, {{LegacyNumericType}} is now deprecated in trunk (to be removed in 7.0), so 
we should migrate away from it ...

But we should maybe take this further, once we get all dimensional values based 
geo queries working well in trunk (e.g. at least {{DimensionalDistanceQuery}} 
and {{DimensionalDistanceRangeQuery}} are still missing?) and deprecate the 
postings based geo queries as well?

> Decouple GeoPointField from NumericType
> ---------------------------------------
>
>                 Key: LUCENE-6930
>                 URL: https://issues.apache.org/jira/browse/LUCENE-6930
>             Project: Lucene - Core
>          Issue Type: Improvement
>            Reporter: Nicholas Knize
>
> {{GeoPointField}} currently relies on {{NumericTokenStream}} to create prefix 
> terms for a GeoPoint using the precision step defined in {{GeoPointField}}. 
> At search time {{GeoPointTermsEnum}} recurses to a max precision that is 
> computed by the Query parameters. This max precision is never the full 
> precision, so creating and indexing the full precision terms is useless and 
> wasteful (it was always a side effect of just using indexing logic from the 
> Numeric type). 
> Furthermore, since the numerical logic always stored high precision terms 
> first, the recursion in {{GeoPointTermsEnum}} required transient memory for 
> storing ranges. By moving the trie logic to its own {{GeoPointTokenStream}} 
> and reversing the term order (such that lower resolution terms are first), 
> the GeoPointTermsEnum can naturally traverse, enabling on-demand creation of 
> PrefixTerms. This will be done in a separate issue.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to