On Mon, 12 May 2025 10:54:05 GMT, Per Minborg <pminb...@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
