RFR: 8223696: java/net/httpclient/MaxStreams.java failed with didn't finish within the time-out [v2]

Daniel Fuchs dfuchs at openjdk.org
Wed Jan 24 09:18:27 UTC 2024


On Wed, 24 Jan 2024 09:11:38 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:

>> The test was occasionally failing with TestNG timeout. When a TestNG test times out, TestNG interrupts the test thread, and starts another test case in a new thread. The timeout is invisible to JTReg, and the timeout failure handlers are not run.
>> 
>> This PR removes the TestNG timeout. It also removes `-ea -esa` from the test command line (the test does not use `assert`), and reduces the amount of synchronization objects used.
>> 
>> I verified that the timeouts are now handled by JTReg. The test still passes.
>
> Daniel Jeliński has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Reintroduce log

Marked as reviewed by dfuchs (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/17536#pullrequestreview-1840859745


More information about the net-dev mailing list