On Tue, 3 Nov 2026 13:08:43 GMT, Daniel Fuchs <[email protected]> wrote:

>> Volkan Yazici has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Fix flushing in `MaxRspTimePropertyTest`
>
> test/jdk/com/sun/net/httpserver/property/IdleIntervalPropertyTest.java line 1:
> 
>> 1: /*
> 
> One comment (not a blocker): It looks like this test will be very long. 
> 40s x 3 + 10 s + 1s ... With different `@test` comments and a bit of luck the 
> bigger offenders will run in parallel, but with a default job concurrency of 
> 4 it's not certain that they will.

I've left only the default execution in `IdleIntervalPropertyTest.java` and 
split the rest into multiple files:

    IdleIntervalPropertyTestZero.java
    IdleIntervalPropertyTestOneWithClockTick.java
    IdleIntervalPropertyTestOne.java
    IdleIntervalPropertyTestNegative.java

See 5f1952ab507.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/31974#discussion_r4467676172

Reply via email to