RFR: 8356658: java/foreign/TestBufferStackStress2.java failed again with junit action timed out [v4]
Jaikiran Pai
jpai at openjdk.org
Mon May 12 11:06:56 UTC 2025
On Mon, 12 May 2025 10:54:05 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:
>
> Address comments
Sorry Per, I didn't notice this in the previous round - I think we should add `@bug 8356658` to the test definition, given the nature of the change to this test. Other than that, the rest of the changes in `6e3f7ee` look good to me.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/25177#issuecomment-2872077214
More information about the core-libs-dev
mailing list