<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">Looks good!</div><div class=""><br class=""></div><div class="">Thanks,</div><div class="">/Staffan</div><div class=""><br class=""></div><div><blockquote type="cite" class=""><div class="">On 21 mars 2016, at 05:10, Harsha Wardhana B <<a href="mailto:harsha.wardhana.b@oracle.com" class="">harsha.wardhana.b@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">
<meta content="text/html; charset=windows-1252" http-equiv="Content-Type" class="">
<div bgcolor="#FFFFFF" text="#000000" class="">
Hello,<br class="">
<br class="">
Please review the new patch located at,<br class="">
<br class="">
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/%7Ehb/8151797/webrev.01/">http://cr.openjdk.java.net/~hb/8151797/webrev.01/</a><br class="">
<br class="">
-Harsha<br class="">
<br class="">
<div class="moz-cite-prefix">On Friday 18 March 2016 05:01 PM,
Staffan Larsen wrote:<br class="">
</div>
<blockquote cite="mid:0F53E9D3-2FC8-4C23-88EB-4F2C0BF0A039@oracle.com" type="cite" class="">
<meta http-equiv="Content-Type" content="text/html;
charset=windows-1252" class="">
You should move the Arrays.stream trigger call to before
the ManagementFactory.getThreadMXBean, otherwise the threads you
get from ThreadMXBean will not match the threads you get from the
ThreadGroups.
<div class=""><br class="">
<div class="">
<blockquote type="cite" class="">
<div class="">On 18 mars 2016, at 12:17, Harsha Wardhana B
<<a moz-do-not-send="true" href="mailto:harsha.wardhana.b@oracle.com" class="">harsha.wardhana.b@oracle.com</a>>
wrote:</div>
<br class="Apple-interchange-newline">
<div class="">
<meta http-equiv="content-type" content="text/html;
charset=windows-1252" class="">
<div bgcolor="#FFFFFF" text="#000000" class=""> Hello All,<br class="">
<br class="">
Please review fix for,<br class="">
<br class="">
Issue : <a moz-do-not-send="true" href="http://jdk-8151797/" class="">JDK-8151797</a> -
java/lang/management/ThreadMXBean/ThreadLists.java :
inconsistent results<br class="">
Webrev : <a moz-do-not-send="true" class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/%7Ehb/8151797/webrev.00/">http://cr.openjdk.java.net/~hb/8151797/webrev.00/</a><br class="">
<br class="">
Root-cause : call-site cleaner thread not started
before counting threads via Thread.getAllStackTraces();.
<br class="">
Fix : Create ThreadMXBean at the beginning of test so
that call-site cleaner thread will be started. <br class="">
<br class="">
Thanks<br class="">
Harsha<br class="">
<br class="">
</div>
</div>
</blockquote>
</div>
<br class="">
</div>
</blockquote>
<br class="">
</div>
</div></blockquote></div><br class=""></body></html>