On Fri, 1 Dec 2023 17:00:48 GMT, Viktor Klang <vkl...@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
