You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-commits@lucene.apache.org by us...@apache.org on 2009/01/11 12:04:56 UTC
svn commit: r733449 -
/lucene/java/trunk/contrib/queries/src/java/org/apache/lucene/search/trie/TrieRangeFilter.java
Author: uschindler
Date: Sun Jan 11 03:04:56 2009
New Revision: 733449
URL: http://svn.apache.org/viewvc?rev=733449&view=rev
Log:
Small optimization because an Object is not needed here
Modified:
lucene/java/trunk/contrib/queries/src/java/org/apache/lucene/search/trie/TrieRangeFilter.java
Modified: lucene/java/trunk/contrib/queries/src/java/org/apache/lucene/search/trie/TrieRangeFilter.java
URL: http://svn.apache.org/viewvc/lucene/java/trunk/contrib/queries/src/java/org/apache/lucene/search/trie/TrieRangeFilter.java?rev=733449&r1=733448&r2=733449&view=diff
==============================================================================
--- lucene/java/trunk/contrib/queries/src/java/org/apache/lucene/search/trie/TrieRangeFilter.java (original)
+++ lucene/java/trunk/contrib/queries/src/java/org/apache/lucene/search/trie/TrieRangeFilter.java Sun Jan 11 03:04:56 2009
@@ -254,15 +254,14 @@
//@Override
public DocIdSet getDocIdSet(IndexReader reader) throws IOException {
final OpenBitSet bits = new OpenBitSet(reader.maxDoc());
- final TermDocs termDocs=reader.termDocs();
+ final TermDocs termDocs = reader.termDocs();
try {
- final int count=splitRange(
+ lastNumberOfTerms=splitRange(
reader,termDocs,bits,
min,trieVariant.TRIE_CODED_NUMERIC_MIN.equals(min),
max,trieVariant.TRIE_CODED_NUMERIC_MAX.equals(max)
);
- lastNumberOfTerms=new Integer(count);
- //System.out.println("Found "+count+" distinct terms in filtered range for field '"+field+"'.");
+ //System.out.println("Found "+lastNumberOfTerms+" distinct terms in filtered range for field '"+field+"'.");
} finally {
termDocs.close();
}
@@ -275,15 +274,14 @@
* This method is not thread safe, be sure to only call it when no query is running!
* @throws IllegalStateException if {@link #getDocIdSet} was not yet executed.
*/
- //@Override
public int getLastNumberOfTerms() {
- if (lastNumberOfTerms==null) throw new IllegalStateException();
- return lastNumberOfTerms.intValue();
+ if (lastNumberOfTerms < 0) throw new IllegalStateException();
+ return lastNumberOfTerms;
}
// members
private final String field,min,max;
private final TrieUtils trieVariant;
private Object minUnconverted,maxUnconverted;
- private Integer lastNumberOfTerms=null;
+ private int lastNumberOfTerms=-1;
}