Skip to content

Commit c75ddd2

Browse files
committed
Fix scaling thread pool test bug
This commit adjusts the expectation for the max number of threads in the scaling thread pool configuration test. The reason that this expectation is incorrect is because we removed the limitation that the number of processors maxes out at 32, instead letting it be the true number of logical processors on the machine. However, when we removed this limitation, this test was never adjusted to reflect the new reality yet it never arose since our tests were not running on machines with incredibly high core counts. Relates #20874
1 parent 1f4d8a0 commit c75ddd2

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

core/src/test/java/org/elasticsearch/threadpool/ScalingThreadPoolTests.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ public void testScalingThreadPoolConfiguration() throws InterruptedException {
5454
maxBasedOnNumberOfProcessors = expectedSize(threadPoolName, processors);
5555
builder.put("processors", processors);
5656
} else {
57-
maxBasedOnNumberOfProcessors = expectedSize(threadPoolName, Math.min(32, Runtime.getRuntime().availableProcessors()));
57+
maxBasedOnNumberOfProcessors = expectedSize(threadPoolName, Runtime.getRuntime().availableProcessors());
5858
}
5959

6060
final int expectedMax;

0 commit comments

Comments
 (0)