On Wed, 24 Jan 2024 09:11:38 GMT, Daniel Jeliński <[email protected]> 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
