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

Posted to commits@lucene.apache.org by us...@apache.org on 2012/02/17 16:58:53 UTC

svn commit: r1245628 - /lucene/dev/trunk/lucene/test-framework/src/java/org/apache/lucene/util/LuceneTestCase.java

Author: uschindler
Date: Fri Feb 17 15:58:53 2012
New Revision: 1245628

URL: http://svn.apache.org/viewvc?rev=1245628&view=rev
Log:
Fix: Don't wrap with single sub MultiReader if already atomic (some tests dont like this).

Modified:
    lucene/dev/trunk/lucene/test-framework/src/java/org/apache/lucene/util/LuceneTestCase.java

Modified: lucene/dev/trunk/lucene/test-framework/src/java/org/apache/lucene/util/LuceneTestCase.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/test-framework/src/java/org/apache/lucene/util/LuceneTestCase.java?rev=1245628&r1=1245627&r2=1245628&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/test-framework/src/java/org/apache/lucene/util/LuceneTestCase.java (original)
+++ lucene/dev/trunk/lucene/test-framework/src/java/org/apache/lucene/util/LuceneTestCase.java Fri Feb 17 15:58:53 2012
@@ -1367,7 +1367,7 @@ public abstract class LuceneTestCase ext
           r = (r instanceof AtomicReader) ?
             new ParallelAtomicReader((AtomicReader) r) :
             new ParallelCompositeReader((CompositeReader) r);
-        } else {
+        } else if (r instanceof CompositeReader) { // only wrap if not already atomic (some tests may fail)
           r = new MultiReader(r);
         }
       }