[
https://issues.apache.org/jira/browse/LUCENE-1530?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12667644#action_12667644
]
Michael McCandless commented on LUCENE-1530:
--------------------------------------------
bq. If nobody complains and the removal of not yet released constructors
without the boolean parameters is ok,
This is perfectly fine. Not yet released APIs are free to change.
> Support inclusive/exclusive for TrieRangeQuery/-Filter, remove default trie
> variant setters/getters
> ---------------------------------------------------------------------------------------------------
>
> Key: LUCENE-1530
> URL: https://issues.apache.org/jira/browse/LUCENE-1530
> Project: Lucene - Java
> Issue Type: New Feature
> Components: contrib/*
> Affects Versions: 2.9
> Reporter: Uwe Schindler
> Assignee: Uwe Schindler
> Fix For: 2.9
>
> Attachments: LUCENE-1530.patch, LUCENE-1530.patch
>
>
> TrieRangeQuery/Filter is missing one thing: Ranges that have exclusive
> bounds. For TrieRangeQuery this may not be important for ranges on long or
> Date (==long) values (because [1..5] is the same like ]0..6[ or ]0..5]). This
> is not so simple for doubles because you must add/substract 1 from the trie
> encoded unsigned long.
> To be conform with the other range queries, I will submit a patch that has
> two additional boolean parameters in the ctors to support inclusive/exclusive
> ranges for both ends. Internally it will be implemented using
> TrieUtils.incrementTrieCoded/decrementTrieCoded() but makes life simplier for
> double ranges (a simple exclusive replacement for the floating point range
> [0.0..1.0] is not possible without having the underlying unsigned long).
> In December, when trie contrib was included (LUCENE-1470), 3 trie variants
> were supplied by TrieUtils. For new APIs a statically configureable default
> Trie variant does not conform to an API we want in Lucene (currently we want
> to deprecate all these static setters/getters). The important thing: It does
> not make code shorter or easier to understand, its more error prone. Before
> release of 2.9 it is a good time to remove the default trie variant and
> always force the parameter in TrieRangeQuery/Filter. It is better to choose
> the variant in the application and do not automatically manage it.
> As Lucene 2.9 was not yet released, I will change the ctors and not preserve
> the old ones.
--
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-unsubscr...@lucene.apache.org
For additional commands, e-mail: java-dev-h...@lucene.apache.org
