You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@lucene.apache.org by rm...@apache.org on 2011/02/14 18:27:41 UTC

svn commit: r1070586 - /lucene/dev/branches/bulkpostings/lucene/src/java/org/apache/lucene/index/codecs/fixed/FixedPostingsReaderImpl.java

Author: rmuir
Date: Mon Feb 14 17:27:41 2011
New Revision: 1070586

URL: http://svn.apache.org/viewvc?rev=1070586&view=rev
Log:
LUCENE-2905: add nocommit

Modified:
    lucene/dev/branches/bulkpostings/lucene/src/java/org/apache/lucene/index/codecs/fixed/FixedPostingsReaderImpl.java

Modified: lucene/dev/branches/bulkpostings/lucene/src/java/org/apache/lucene/index/codecs/fixed/FixedPostingsReaderImpl.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/bulkpostings/lucene/src/java/org/apache/lucene/index/codecs/fixed/FixedPostingsReaderImpl.java?rev=1070586&r1=1070585&r2=1070586&view=diff
==============================================================================
--- lucene/dev/branches/bulkpostings/lucene/src/java/org/apache/lucene/index/codecs/fixed/FixedPostingsReaderImpl.java (original)
+++ lucene/dev/branches/bulkpostings/lucene/src/java/org/apache/lucene/index/codecs/fixed/FixedPostingsReaderImpl.java Mon Feb 14 17:27:41 2011
@@ -1228,6 +1228,8 @@ public class FixedPostingsReaderImpl ext
 
     @Override
     public int fill() throws IOException {
+      // nocommit: keep freqs "one step behind" docs? this could avoid skipBlock() 
+      // calls for low-freq terms (e.g. termrangequery/wildcard) where omitTF is off.
       final int ret = docs.fill();
       freqs.skipBlock();
       return ret;