RFR: 8356658: java/foreign/TestBufferStackStress2.java failed again with junit action timed out [v6]

Jaikiran Pai jpai at openjdk.org
Mon May 12 14:54:52 UTC 2025


On Mon, 12 May 2025 14:41:27 GMT, Per Minborg <pminborg at openjdk.org> wrote:

>> This PR proposes to add a safety net around closing the executor. Apparently, in some rare configuration, the VT is not run/notified correctly.
>> 
>> Not completing the test for such a configuration is unlikely to mask potential issues that this test is supposed to reveal.
>
> Per Minborg has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Remove delimiter between JBS issues

Looks good to me and thank you for the updates.

If this was failing every single time in tier5, then it might be useful to run these changes once in tier5 in our CI before integrating.

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

Marked as reviewed by jpai (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/25177#pullrequestreview-2833538419


More information about the core-libs-dev mailing list