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:44:05 UTC
svn commit: r1619213 - in /lucene/dev/branches/branch_4x: ./ lucene/
lucene/test-framework/
lucene/test-framework/src/java/org/apache/lucene/util/LuceneTestCase.java
Author: simonw
Date: Wed Aug 20 19:44:05 2014
New Revision: 1619213
URL: http://svn.apache.org/r1619213
Log:
LUCENE-5896: Use local random instance rather than global in LuceneTestCase
Modified:
lucene/dev/branches/branch_4x/ (props changed)
lucene/dev/branches/branch_4x/lucene/ (props changed)
lucene/dev/branches/branch_4x/lucene/test-framework/ (props changed)
lucene/dev/branches/branch_4x/lucene/test-framework/src/java/org/apache/lucene/util/LuceneTestCase.java
Modified: lucene/dev/branches/branch_4x/lucene/test-framework/src/java/org/apache/lucene/util/LuceneTestCase.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_4x/lucene/test-framework/src/java/org/apache/lucene/util/LuceneTestCase.java?rev=1619213&r1=1619212&r2=1619213&view=diff
==============================================================================
--- lucene/dev/branches/branch_4x/lucene/test-framework/src/java/org/apache/lucene/util/LuceneTestCase.java (original)
+++ lucene/dev/branches/branch_4x/lucene/test-framework/src/java/org/apache/lucene/util/LuceneTestCase.java Wed Aug 20 19:44:05 2014
@@ -941,8 +941,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);
@@ -1034,7 +1034,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);
@@ -1104,7 +1104,7 @@ public abstract class LuceneTestCase ext
flushByRAM = false;
break;
case EITHER:
- flushByRAM = random().nextBoolean();
+ flushByRAM = r.nextBoolean();
break;
default:
throw new AssertionError();