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

2023-12-05 Thread Alan Bateman
On Tue, 5 Dec 2023 14:19:34 GMT, Viktor Klang wrote: > Do you want me to make this change as a part of this PR or does it make more > sense to get this one integrated first? I think it's okay to leave it to another time, I assume these tests be updated a few times before the feature becomes pe

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

2023-12-05 Thread Viktor Klang
On Tue, 5 Dec 2023 14:07:53 GMT, Alan Bateman wrote: >> Viktor Klang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Removing extraneous built-in gatherer test output > > test/jdk/java/util/stream/GatherersMapConcurrentTest.java line 106

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

2023-12-05 Thread Alan Bateman
On Mon, 4 Dec 2023 14:25:05 GMT, Viktor Klang wrote: >> Splits GatherersTest out to distinct files per built-in gatherer >> >> Fixes a test ordering issue for mapConcurrent which placed GatherersTest on >> the ProblemList. >> >> Adding increased maxOutputSize for Gatherer-related tests to impr

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

2023-12-05 Thread Viktor Klang
On Mon, 4 Dec 2023 14:25:05 GMT, Viktor Klang wrote: >> Splits GatherersTest out to distinct files per built-in gatherer >> >> Fixes a test ordering issue for mapConcurrent which placed GatherersTest on >> the ProblemList. >> >> Adding increased maxOutputSize for Gatherer-related tests to impr

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

2023-12-04 Thread Viktor Klang
> Splits GatherersTest out to distinct files per built-in gatherer > > Fixes a test ordering issue for mapConcurrent which placed GatherersTest on > the ProblemList. > > Adding increased maxOutputSize for Gatherer-related tests to improve > debuggability. > > Lowering the composition threshold