On Wed, 2 Apr 2025 11:10:40 GMT, Daniel Fuchs <[email protected]> wrote:
>> Volkan Yazici has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Apply review suggestions
>
> src/java.net.http/share/classes/jdk/internal/net/http/HttpConnection.java
> line 100:
>
>> 98: * {@link PlainHttpConnection} it wraps will share the same label.
>> As a
>> 99: * result, compared to {@link #id}, this label does not give a total
>> order
>> 100: * among instances.
>
> Suggestion:
>
> /**
> * A label to identify the connection.
> * <p>
> * This label helps with associating multiple components participating in
> a
> * connection. For instance, an {@link AsyncSSLConnection} and the
> * {@link PlainHttpConnection} it wraps will share the same label.
Implemented in 42ff0fc4a83cd33f1a35e4a2b3e7ce5c25ea39cc.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24154#discussion_r2028746945
