Hi Daniil,
LGTM++
Thanks,
Serguei
On 4/17/20 14:14, Chris Plummer wrote:
Looks good.
Chris
On 4/17/20 1:03 PM, Daniil Titov wrote:
Please review the change that fixes intermittent failure of
java/lang/management/ThreadMXBean/MaxDepthForThreadInfoTest.java
As David noticed (thank you, David, for this analysis) there is no
guarantee that all threads found by getAllThreadIds() are still alive
by the time we call getThreadInfo() so we have to allow for null
array entries.
Testing: Mach5 tests with Graal on passed 300 times.
[1] http://cr.openjdk.java.net/~dtitov/8231585/webrev.01/
[2] https://bugs.openjdk.java.net/browse/JDK-8231585
Best regards,
Daniil