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 2012/05/23 20:16:21 UTC

svn commit: r1341967 - /lucene/dev/branches/lucene4055/lucene/core/src/java/org/apache/lucene/index/SegmentMerger.java

Author: rmuir
Date: Wed May 23 18:16:21 2012
New Revision: 1341967

URL: http://svn.apache.org/viewvc?rev=1341967&view=rev
Log:
LUCENE-4055: nuke obselete nocommit, expand on note

Modified:
    lucene/dev/branches/lucene4055/lucene/core/src/java/org/apache/lucene/index/SegmentMerger.java

Modified: lucene/dev/branches/lucene4055/lucene/core/src/java/org/apache/lucene/index/SegmentMerger.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/lucene4055/lucene/core/src/java/org/apache/lucene/index/SegmentMerger.java?rev=1341967&r1=1341966&r2=1341967&view=diff
==============================================================================
--- lucene/dev/branches/lucene4055/lucene/core/src/java/org/apache/lucene/index/SegmentMerger.java (original)
+++ lucene/dev/branches/lucene4055/lucene/core/src/java/org/apache/lucene/index/SegmentMerger.java Wed May 23 18:16:21 2012
@@ -55,8 +55,7 @@ final class SegmentMerger {
   private final MergeState mergeState = new MergeState();
   private final FieldInfos.Builder fieldInfosBuilder;
 
-  // nocommit nuke name since SI has it.... but Directory is
-  // NOT the same!!
+  // note, just like in codec apis Directory 'dir' is NOT the same as segmentInfo.dir!!
   SegmentMerger(SegmentInfo segmentInfo, InfoStream infoStream, Directory dir, int termIndexInterval,
                 MergeState.CheckAbort checkAbort, PayloadProcessorProvider payloadProcessorProvider,
                 FieldInfos.FieldNumbers fieldNumbers, IOContext context) {