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/17 13:22:50 UTC
svn commit: r1071593 - in
/lucene/dev/trunk/lucene/src/java/org/apache/lucene/index/codecs/sep:
SepPostingsReaderImpl.java SepPostingsWriterImpl.java
Author: rmuir
Date: Thu Feb 17 12:22:49 2011
New Revision: 1071593
URL: http://svn.apache.org/viewvc?rev=1071593&view=rev
Log:
LUCENE-2905: don't write abs skip pointer until we have to, the block might be all low-freq terms
Modified:
lucene/dev/trunk/lucene/src/java/org/apache/lucene/index/codecs/sep/SepPostingsReaderImpl.java
lucene/dev/trunk/lucene/src/java/org/apache/lucene/index/codecs/sep/SepPostingsWriterImpl.java
Modified: lucene/dev/trunk/lucene/src/java/org/apache/lucene/index/codecs/sep/SepPostingsReaderImpl.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/src/java/org/apache/lucene/index/codecs/sep/SepPostingsReaderImpl.java?rev=1071593&r1=1071592&r2=1071593&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/src/java/org/apache/lucene/index/codecs/sep/SepPostingsReaderImpl.java (original)
+++ lucene/dev/trunk/lucene/src/java/org/apache/lucene/index/codecs/sep/SepPostingsReaderImpl.java Thu Feb 17 12:22:49 2011
@@ -242,7 +242,7 @@ public class SepPostingsReaderImpl exten
}
//System.out.println(" skipFP=" + termState.skipFP);
} else if (isFirstTerm) {
- termState.skipFP = termState.bytesReader.readVLong();
+ termState.skipFP = 0;
}
}
Modified: lucene/dev/trunk/lucene/src/java/org/apache/lucene/index/codecs/sep/SepPostingsWriterImpl.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/src/java/org/apache/lucene/index/codecs/sep/SepPostingsWriterImpl.java?rev=1071593&r1=1071592&r2=1071593&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/src/java/org/apache/lucene/index/codecs/sep/SepPostingsWriterImpl.java (original)
+++ lucene/dev/trunk/lucene/src/java/org/apache/lucene/index/codecs/sep/SepPostingsWriterImpl.java Thu Feb 17 12:22:49 2011
@@ -289,12 +289,8 @@ public final class SepPostingsWriterImpl
}
lastSkipFP = skipFP;
} else if (isFirstTerm) {
- // TODO: this is somewhat wasteful; eg if no terms in
- // this block will use skip data, we don't need to
- // write this:
- final long skipFP = skipOut.getFilePointer();
- indexBytesWriter.writeVLong(skipFP);
- lastSkipFP = skipFP;
+ // lazily write an absolute delta if a term in this block requires skip data.
+ lastSkipFP = 0;
}
lastDocID = 0;