RFR: 8330814: Cleanups for KeepAliveCache tests [v5]

Christoph Langer clanger at openjdk.org
Wed May 1 20:45:29 UTC 2024


On Tue, 30 Apr 2024 15:07:52 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> Christoph Langer has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains seven additional commits since the last revision:
>> 
>>  - Merge branch 'master' into keepalivetests
>>  - Fix two URLs
>>  - Merge branch 'master' into keepalivetests
>>  - Jaikiran's comments
>>  - Update test/jdk/sun/net/www/http/KeepAliveCache/B5045306.java
>>    
>>    Co-authored-by: Andrey Turbanov <turbanoff at gmail.com>
>>  - Copyright year
>>  - JDK-8330814
>
> test/jdk/sun/net/www/http/KeepAliveCache/B8291637.java line 153:
> 
>> 151:         }
>> 152:         System.out.println("Read " + count);
>> 153:         try {
> 
> Maybe the `try {` line should be moved just after `server.start()`

good point. I enhanced this a little bit more. 😄

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18884#discussion_r1586803643


More information about the net-dev mailing list