On Fri, 30 Jan 2026 10:58:33 GMT, Daniel Jeliński <[email protected]> wrote:
>> Daniel Fuchs has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Review feedback
>
> test/jdk/java/net/httpclient/CancelRequestTest.java line 789:
>
>> 787: requestLatch, t.getRequestURI());
>> 788: try {
>> 789: requestLatch.await();
>
> Can we remove the Thread.sleep above now?
I'd rather not. The thread.sleep() is here to introduce some more entropy on
where the client might be when the cancellation becomes effective. There's more
chance that it won't have received all the data.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29415#discussion_r2745758838
