Re: RFR: 8389617: Clean up the Http2TestServerConnection in httpclient test library and introduce a way to access the underlying exchange from a HttpTestExchange [v3]

[Jaikiran Pai]

On Wed, 5 Aug 2026 09:00:18 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:

> May writeLoopThread be null here?

I've updated the PR to make sure that when `close()` is called on the 
connection, the write/read Threads are never null.

> What if close() is called from within the writeLoopThread?

Good question. I hadn't considered that. In theory a calling close() from the 
`writeLoopThread` would have deadlocked. I've now updated the PR to add a check 
to make sure this doesn't happen.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/32173#discussion_r3720403711