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