RFR: 8350279: HttpClient: Add a new HttpResponse method to identify connections [v4]
Jaikiran Pai
jpai at openjdk.org
Thu Mar 27 14:16:26 UTC 2025
On Thu, 27 Mar 2025 13:55:34 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:
> The only use of HttpConnection.id is for ordering - so that connections can be placed in a ConcurrentSkipListSet.
I wasn't aware of that. Now that you mentioned it, I looked up the code which uses the Set to store these connections. And from what I can see, the order is only used during the closing of a HttpClient, to close these opened connections in that specific order. Did I miss any other usages of the order?
In any case, now that you corrected me about the usage of the `id` order, I agree that having the comment on the `id` field is fine and only remove it from the paragraph here.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24154#discussion_r2016769197
More information about the net-dev
mailing list