RFR: 8376308: java/net/httpclient/CancelRequestTest.java fails intermittently with "Expected CancellationException not received" [v2]

Daniel Fuchs dfuchs at openjdk.org
Fri Jan 30 11:15:05 UTC 2026


On Fri, 30 Jan 2026 10:58:33 GMT, Daniel Jeliński <djelinski at openjdk.org> 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


More information about the net-dev mailing list