On Tue, 11 Feb 2025 11:46:44 GMT, Volkan Yazici <vyaz...@openjdk.org> wrote:
>> Fixes stream cancellation in `Http2Connection::putStream`. See comments in >> the ticket for details on the issue. `tier1,2` results are attached to the >> ticket too. > > Volkan Yazici has updated the pull request incrementally with two additional > commits since the last revision: > > - Add current issue to the `@bug` list in `AsyncExecutorShutdown` > - Wrap the cause in an `IOException` > > This is okay, because `AsyncExecutorShutdown` walks > the causal chain while searching for a certain > exception footprint. Thank you for these updates. This looks good to me. Since it was the `test/jdk/java/net/httpclient/AsyncExecutorShutdown.java` test which caught this intermittent failure, it's the right thing to update the `@bug` of this test to refer to this issue. ------------- Marked as reviewed by jpai (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/23540#pullrequestreview-2608490003
