Re: RFR: 8321124: java/util/stream/GatherersTest.java times out [v2]

2023-12-04 Thread Alan Bateman
On Sun, 3 Dec 2023 09:50:53 GMT, Alan Bateman wrote: >> test/jdk/java/util/stream/BuiltInGatherersTest.java line 331: >> >>> 329: case Integer n when n == >>> config.streamSize - 1 -> { >>> 330: awaitSensibly(firstReady

Re: RFR: 8321124: java/util/stream/GatherersTest.java times out [v2]

2023-12-04 Thread Viktor Klang
On Sun, 3 Dec 2023 09:20:50 GMT, Alan Bateman wrote: >> test/jdk/java/util/stream/BuiltInGatherersTest.java line 257: >> >>> 255: // Test cancellation after exception during processing >>> 256: // Only use reasonably sized streams to avoid excessive thread >>> creation >>> 257:

Re: RFR: 8321124: java/util/stream/GatherersTest.java times out [v2]

2023-12-04 Thread Viktor Klang
On Sun, 3 Dec 2023 09:26:45 GMT, Alan Bateman wrote: >> Viktor Klang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Only run the cancellation tests for mapConcurrent for the shorter streams >> to conserve resources > > test/jdk/java/ut

Re: RFR: 8321124: java/util/stream/GatherersTest.java times out [v2]

2023-12-03 Thread Alan Bateman
On Fri, 1 Dec 2023 22:16:45 GMT, Viktor Klang 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

Re: RFR: 8321124: java/util/stream/GatherersTest.java times out [v2]

2023-12-03 Thread Alan Bateman
On Fri, 1 Dec 2023 22:11:05 GMT, Viktor Klang wrote: >> Viktor Klang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Only run the cancellation tests for mapConcurrent for the shorter streams >> to conserve resources > > test/jdk/java/ut

Re: RFR: 8321124: java/util/stream/GatherersTest.java times out [v2]

2023-12-01 Thread Viktor Klang
On Fri, 1 Dec 2023 22:06:14 GMT, Viktor Klang 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

Re: RFR: 8321124: java/util/stream/GatherersTest.java times out [v2]

2023-12-01 Thread Viktor Klang
> 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