RFR: 8321124: java/util/stream/GatherersTest.java times out
Viktor Klang
vklang at openjdk.org
Fri Dec 1 17:06:32 UTC 2023
On Fri, 1 Dec 2023 17:00:48 GMT, Viktor Klang <vklang at openjdk.org> wrote:
> Renames GatherersTest to BuiltInGatherersTest for easier deduplication of GathererTest.
>
> Fixes a test ordering issue in testMapConcurrentAPIandContract().
>
> Adding increased maxOutputSize for Gatherer-related tests to improve debuggability.
>
> Lowering the composition threshold of GathererTest.testMassivelyComposedGatherers to 256 to avoid SOE on low-specc:ed machines.
test/jdk/java/util/stream/BuiltInGatherersTest.java line 331:
> 329: case Integer n when n == config.streamSize - 1 -> {
> 330: awaitSensibly(firstReady);
> 331: while(tasksWaiting.getQueueLength() < tasksToCancel) {
@AlanBateman This is the only part of this I really don't like. Any better suggestion as to "wait for N things waiting" in a test? 🤔
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16928#discussion_r1412368710
More information about the core-libs-dev
mailing list