You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by mi...@apache.org on 2010/12/22 12:32:30 UTC
svn commit: r1051850 -
/lucene/dev/branches/bulkpostings/lucene/src/java/org/apache/lucene/index/codecs/sep/SepPostingsWriterImpl.java
Author: mikemccand
Date: Wed Dec 22 11:32:30 2010
New Revision: 1051850
URL: http://svn.apache.org/viewvc?rev=1051850&view=rev
Log:
upgrade TODO to nocommit
Modified:
lucene/dev/branches/bulkpostings/lucene/src/java/org/apache/lucene/index/codecs/sep/SepPostingsWriterImpl.java
Modified: lucene/dev/branches/bulkpostings/lucene/src/java/org/apache/lucene/index/codecs/sep/SepPostingsWriterImpl.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/bulkpostings/lucene/src/java/org/apache/lucene/index/codecs/sep/SepPostingsWriterImpl.java?rev=1051850&r1=1051849&r2=1051850&view=diff
==============================================================================
--- lucene/dev/branches/bulkpostings/lucene/src/java/org/apache/lucene/index/codecs/sep/SepPostingsWriterImpl.java (original)
+++ lucene/dev/branches/bulkpostings/lucene/src/java/org/apache/lucene/index/codecs/sep/SepPostingsWriterImpl.java Wed Dec 22 11:32:30 2010
@@ -172,7 +172,7 @@ public final class SepPostingsWriterImpl
public void startDoc(int docID, int termDocFreq) throws IOException {
if (firstDoc) {
- // TODO: we are writing absolute file pointers below,
+ // nocommit: we are writing absolute file pointers below,
// which is wasteful. It'd be better compression to
// write the "baseline" into each indexed term, then
// write only the delta here.
@@ -183,7 +183,8 @@ public final class SepPostingsWriterImpl
docOut.writeVLong(payloadStart);
}
}
- // nocommit -- only write if docFreq > skipInterval?
+ // nocommit -- only write if docFreq > skipInterval
+ // nocommit -- use delta not abs
docOut.writeVLong(skipOut.getFilePointer());
firstDoc = false;
}