On Tue, 28 Apr 2026 11:12:30 GMT, Mikhail Yankelevich 
<[email protected]> wrote:

>> * fully automated the test
>> * removed the race condition
>> * client on a thread and server on a thread options are now run together 
>> automatically
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Mikhail Yankelevich has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Daniel's comments

test/jdk/sun/net/www/protocol/https/HttpsURLConnection/CloseKeepAliveCached.java
 line 102:

> 100:      * Is the server ready to serve?
> 101:      */
> 102:     final static CountDownLatch serverReady = new CountDownLatch(1);

Suggestion:

    static final CountDownLatch serverReady = new CountDownLatch(1);

test/jdk/sun/net/www/protocol/https/HttpsURLConnection/CloseKeepAliveCached.java
 line 104:

> 102:     final static CountDownLatch serverReady = new CountDownLatch(1);
> 103: 
> 104:     volatile private SSLServerSocket sslServerSocket = null;

Suggestion:

    private volatile SSLServerSocket sslServerSocket = null;

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23469#discussion_r3155620155
PR Review Comment: https://git.openjdk.org/jdk/pull/23469#discussion_r3155622971

Reply via email to