You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by si...@apache.org on 2014/08/20 21:36:20 UTC
svn commit: r1619211 -
/lucene/dev/trunk/lucene/test-framework/src/java/org/apache/lucene/util/LuceneTestCase.java
Author: simonw
Date: Wed Aug 20 19:36:20 2014
New Revision: 1619211
URL: http://svn.apache.org/r1619211
Log:
LUCENE-5896: Use local random instance rather than global in LuceneTestCase
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=1619211&r1=1619210&r2=1619211&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 Wed Aug 20 19:36:20 2014
@@ -892,8 +892,8 @@ public abstract class LuceneTestCase ext
if (r.nextBoolean()) {
c.setMergeScheduler(new SerialMergeScheduler());
} else if (rarely(r)) {
- int maxThreadCount = TestUtil.nextInt(random(), 1, 4);
- int maxMergeCount = TestUtil.nextInt(random(), maxThreadCount, maxThreadCount + 4);
+ int maxThreadCount = TestUtil.nextInt(r, 1, 4);
+ int maxMergeCount = TestUtil.nextInt(r, maxThreadCount, maxThreadCount + 4);
ConcurrentMergeScheduler cms = new ConcurrentMergeScheduler();
cms.setMaxMergesAndThreads(maxMergeCount, maxThreadCount);
c.setMergeScheduler(cms);
@@ -975,7 +975,7 @@ public abstract class LuceneTestCase ext
mergePolicy.setNoCFSRatio(r.nextBoolean() ? 1.0 : 0.0);
}
- if (rarely()) {
+ if (rarely(r)) {
mergePolicy.setMaxCFSSegmentSizeMB(0.2 + r.nextDouble() * 2.0);
} else {
mergePolicy.setMaxCFSSegmentSizeMB(Double.POSITIVE_INFINITY);
@@ -1045,7 +1045,7 @@ public abstract class LuceneTestCase ext
flushByRAM = false;
break;
case EITHER:
- flushByRAM = random().nextBoolean();
+ flushByRAM = r.nextBoolean();
break;
default:
throw new AssertionError();