<html>
<head>
<meta http-equiv="Content-Type" content="text/html;
charset=windows-1252">
</head>
<body>
<p>I didn't want to touch unrelated files and spread this fix
outside vmTestbase/nsk/monitoring/ThreadMBean package. I wanted
just to merge folders to don't confuse people by layout.<br>
</p>
<p>The ThreadMBean is mentioned also in <br>
</p>
<p><tt>test/jdk/sun/management/jmxremote/bootstrap/</tt></p>
<p> files and <br>
</p>
<p><tt>src/jdk.management.agent/share/conf/management.properties</tt></p>
<p>Also sometimes just MBean used to point to corresponding MXBean
in comments. (Might it is a correct to use MBean there)<br>
</p>
<p>So, let just change ThreadMBean in nsk/monitoring/stress/thread/
and stop updating comments on this.</p>
<p>BTW, I will change summary to <br>
</p>
<p>8242295: Change ThreadMBean in vmTestbase/nsk/monitoring to
ThreadMXBean</p>
<p>to better describe changes<br>
</p>
<p>Leonid<br>
</p>
<div class="moz-cite-prefix">On 4/7/20 4:06 PM, Igor Ignatyev wrote:<br>
</div>
<blockquote type="cite"
cite="mid:DEF78A6E-73DF-4C11-92C3-5B530F05424C@oracle.com">great!
what about nsk/monitoring/stress/thread/ ? they all have lines
'and states gotten via the ThreadMBean interface.'
(strace010.java has 3 occurrences, other files just one)
<div class=""><br class="">
</div>
<div class="">-- Igor <br class="">
<div><br class="">
<blockquote type="cite" class="">
<div class="">On Apr 7, 2020, at 3:56 PM, Leonid Mesnik <<a
href="mailto:leonid.mesnik@oracle.com" class=""
moz-do-not-send="true">leonid.mesnik@oracle.com</a>>
wrote:</div>
<br class="Apple-interchange-newline">
<div class="">
<div class="">
<p class="">Sure<br class="">
</p>
<p class="">I've updated ThreadMBean to ThreadMXBean in
test descriptions for ThreadMXBean tests .</p>
<p class=""><a
href="http://cr.openjdk.java.net/~lmesnik/8242295/webrev.01/"
class="" moz-do-not-send="true">http://cr.openjdk.java.net/~lmesnik/8242295/webrev.01/</a></p>
<p class="">Leonid<br class="">
</p>
<div class="moz-cite-prefix">On 4/7/20 3:00 PM, Igor
Ignatyev wrote:<br class="">
</div>
<blockquote type="cite"
cite="mid:15ED9DD2-0448-433A-B5CA-0D8B479E2BD4@oracle.com"
class=""> Hi Leonid,
<div class=""><br class="">
</div>
<div class="">looks good and trivial to me. one
question, will it also make sense to replace
ThreadMBean w/ ThreadMXBean in test descriptions,
e.g. at L#33 of
test/hotspot/jtreg/vmTestbase/nsk/monitoring/ThreadMXBean/isCurrentThreadCpuTimeSupported/curthcputime001/TestDescription.java?</div>
<div class=""><br class="">
</div>
<div class="">Thanks,</div>
<div class="">-- Igor<br class="">
<div class=""><br class="">
<blockquote type="cite" class="">
<div class="">On Apr 7, 2020, at 2:46 PM, Leonid
Mesnik <<a
href="mailto:leonid.mesnik@oracle.com"
class="" moz-do-not-send="true">leonid.mesnik@oracle.com</a>>
wrote:</div>
<br class="Apple-interchange-newline">
<div class="">
<div class="">
<div class="">Hi</div>
<div class="">Could you please review
following fix which just moves content
of ThreadMBean back into ThreadMXBean. </div>
<div class=""><br class="">
</div>
<div class="">webrev: <a
href="http://cr.openjdk.java.net/~lmesnik/8242295/webrev.00/"
class="" moz-do-not-send="true">http://cr.openjdk.java.net/~lmesnik/8242295/webrev.00/</a></div>
<div class="">bug: <a
href="https://bugs.openjdk.java.net/browse/JDK-8242295"
class="" moz-do-not-send="true">https://bugs.openjdk.java.net/browse/JDK-8242295</a></div>
</div>
</div>
</blockquote>
</div>
<br class="">
</div>
</blockquote>
</div>
</div>
</blockquote>
</div>
<br class="">
</div>
</blockquote>
</body>
</html>