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/02/21 21:44:16 UTC
svn commit: r1291999 -
/lucene/dev/trunk/lucene/test-framework/src/java/org/apache/lucene/util/LuceneTestCase.java
Author: rmuir
Date: Tue Feb 21 20:44:16 2012
New Revision: 1291999
URL: http://svn.apache.org/viewvc?rev=1291999&view=rev
Log:
don't wrap already-atomic readers in crazy hierarchies while Uwe is visiting relatives
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=1291999&r1=1291998&r2=1291999&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 Tue Feb 21 20:44:16 2012
@@ -1351,6 +1351,13 @@ public abstract class LuceneTestCase ext
public static IndexReader maybeWrapReader(IndexReader r) throws IOException {
// TODO: remove this, and fix those tests to wrap before putting slow around:
final boolean wasOriginallyAtomic = r instanceof AtomicReader;
+
+ if (wasOriginallyAtomic) {
+ // TODO: investigate purging etc of tests making top-level fieldcaches,
+ // something is up if they get a crazy hierarchy
+ return r;
+ }
+
if (rarely()) {
for (int i = 0, c = random.nextInt(6)+1; i < c; i++) {
switch(random.nextInt(4)) {