[jira] Commented: (LUCENE-1505) Change contrib/spatial to use trie's NumericUtils, and remove NumberUtils

["Uwe Schindler (JIRA)" <ji...@apache.org>]

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

Uwe Schindler commented on LUCENE-1505:
---------------------------------------

Patch looks good. The best is, that you can now use the double FieldCache which should speed up the LocalLucene queries and needs less memory.

By the way: I was also tending to add these convenience methods to NumericUtils :-) For completeness, shoudl we also add them for the ones with the shift value at the end? an char[]? I was reluctant to do this.

I think the easy conversion methods to/from String withou shift would be enough.

> Change contrib/spatial to use trie's NumericUtils, and remove NumberUtils
> -------------------------------------------------------------------------
>
>                 Key: LUCENE-1505
>                 URL: https://issues.apache.org/jira/browse/LUCENE-1505
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: contrib/spatial
>            Reporter: Ryan McKinley
>            Assignee: Michael McCandless
>             Fix For: 2.9
>
>         Attachments: LUCENE-1505.patch
>
>
> Currently spatial contrib includes a copy of NumberUtils from solr (otherwise it would depend on solr)
> Once LUCENE-1496 is sorted out, this copy should be removed.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: java-dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: java-dev-help@lucene.apache.org