RFR: 8355370: Include server name in HTTP test server thread names to improve diagnostics [v2]
    Volkan Yazici 
    vyazici at openjdk.org
       
    Thu Apr 24 07:24:37 UTC 2025
    
    
  
On Wed, 23 Apr 2025 14:58:45 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:
>> 1. What is wrong with `HttpTestServer::close` calling `HttpTestServer::stop`, which is already implemented as `impl.close()` by `Http2TestServerImpl`?
>> 2. Shouldn't we better remove `implements AutoCloseable` and `close()` from `Http2TestServerImpl`?
>
> 1. If the impl is AutoCloseable then close() should call close().
> 2. I would actually prefer to make all implementations AutoCloseable, but that's not the point of this PR.
Implemented changes as requested.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24822#discussion_r2057718592
    
    
More information about the net-dev
mailing list