RFR: 8304065: HttpServer.stop should terminate immediately if no exchanges are in progress [v4]

Mikhail Yankelevich myankelevich at openjdk.org
Fri May 23 08:57:54 UTC 2025


On Fri, 23 May 2025 08:50:11 GMT, Mark Sheppard <msheppar at openjdk.org> wrote:

>> Mikhail Yankelevich has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Suggestions from cr
>>   
>>   Co-authored-by: Daniel Fuchs <67001856+dfuch at users.noreply.github.com>
>
> src/jdk.httpserver/share/classes/sun/net/httpserver/ServerImpl.java line 227:
> 
>> 225:     }
>> 226: 
>> 227:     public final boolean isFinishing() {
> 
> encapsulating the server state in a CountDoenLatch is obfuscated semantics

Yes, but this is an existing function and it is used in other classes. I didn't change the functionality

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25333#discussion_r2104143995


More information about the net-dev mailing list