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 2012/05/23 22:20:10 UTC

svn commit: r1342023 - in /lucene/dev/branches/lucene4055/lucene: core/src/test/org/apache/lucene/index/TestIndexWriter.java misc/src/java/org/apache/lucene/index/IndexSplitter.java

Author: mikemccand
Date: Wed May 23 20:20:10 2012
New Revision: 1342023

URL: http://svn.apache.org/viewvc?rev=1342023&view=rev
Log:
LUCENE-4055: more nocommits gone...

Modified:
    lucene/dev/branches/lucene4055/lucene/core/src/test/org/apache/lucene/index/TestIndexWriter.java
    lucene/dev/branches/lucene4055/lucene/misc/src/java/org/apache/lucene/index/IndexSplitter.java

Modified: lucene/dev/branches/lucene4055/lucene/core/src/test/org/apache/lucene/index/TestIndexWriter.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/lucene4055/lucene/core/src/test/org/apache/lucene/index/TestIndexWriter.java?rev=1342023&r1=1342022&r2=1342023&view=diff
==============================================================================
--- lucene/dev/branches/lucene4055/lucene/core/src/test/org/apache/lucene/index/TestIndexWriter.java (original)
+++ lucene/dev/branches/lucene4055/lucene/core/src/test/org/apache/lucene/index/TestIndexWriter.java Wed May 23 20:20:10 2012
@@ -1528,13 +1528,6 @@ public class TestIndexWriter extends Luc
     for (IndexReader r : r0.getSequentialSubReaders()) {
       SegmentInfoPerCommit s = ((SegmentReader) r).getSegmentInfo();
       assertFalse(((SegmentReader) r).getFieldInfos().hasVectors());
-      // nocommit
-      /*
-      Set<String> files = new HashSet<String>();
-      s.getCodec().termVectorsFormat().files(s, files);
-      List<String> filesExisting = SegmentInfo.findMatchingFiles(s.name, dir, files);
-      assertTrue(filesExisting.isEmpty());
-      */
     }
     
     r0.close();

Modified: lucene/dev/branches/lucene4055/lucene/misc/src/java/org/apache/lucene/index/IndexSplitter.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/lucene4055/lucene/misc/src/java/org/apache/lucene/index/IndexSplitter.java?rev=1342023&r1=1342022&r2=1342023&view=diff
==============================================================================
--- lucene/dev/branches/lucene4055/lucene/misc/src/java/org/apache/lucene/index/IndexSplitter.java (original)
+++ lucene/dev/branches/lucene4055/lucene/misc/src/java/org/apache/lucene/index/IndexSplitter.java Wed May 23 20:20:10 2012
@@ -149,7 +149,6 @@ public class IndexSplitter {
                                             info.getDocStoreSegment(), info.getDocStoreIsCompoundFile(), info.getNormGen(), info.getUseCompoundFile(),
                                             info.getCodec(), info.getDiagnostics(), info.attributes());
       destInfos.add(new SegmentInfoPerCommit(newInfo, infoPerCommit.getDelCount(), infoPerCommit.getDelGen()));
-      // nocommit is this right...?
       // now copy files over
       Collection<String> files = infoPerCommit.files();
       for (final String srcName : files) {