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