RFR: 8299419: Thread.sleep(millis) may throw OOME [v2]

[Viktor Klang]

On Wed, 11 Sep 2024 07:00:25 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> If these are just methods that might be seen in a stackdump then we still need `ThreadSleepEvent.isEnabled` and `ThreadSleepEvent.<init>`.
>
>> If these are just methods that might be seen in a stackdump then we still need `ThreadSleepEvent.isEnabled` and `ThreadSleepEvent.<init>`.
> 
> I think minimal here is to just drop isTurnedOn from lists in these tests. From what I can tell, the tests are sampling in a loop and can tolerate up to a limit. Hopefully someone will step up some day and replace them.

@AlanBateman @dholmes-ora Ok, that's fair enough. I have replaced the commit which dropped all three in the trace to just dropping the isTurnedOn frame.

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

PR Comment: https://git.openjdk.org/jdk/pull/20923#issuecomment-2343431165