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)
-- Igor > On Apr 7, 2020, at 3:56 PM, Leonid Mesnik <[email protected]> wrote: > > Sure > > I've updated ThreadMBean to ThreadMXBean in test descriptions for > ThreadMXBean tests . > > http://cr.openjdk.java.net/~lmesnik/8242295/webrev.01/ > <http://cr.openjdk.java.net/~lmesnik/8242295/webrev.01/> > Leonid > > On 4/7/20 3:00 PM, Igor Ignatyev wrote: >> Hi Leonid, >> >> 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? >> >> Thanks, >> -- Igor >> >>> On Apr 7, 2020, at 2:46 PM, Leonid Mesnik <[email protected] >>> <mailto:[email protected]>> wrote: >>> >>> Hi >>> Could you please review following fix which just moves content of >>> ThreadMBean back into ThreadMXBean. >>> >>> webrev: http://cr.openjdk.java.net/~lmesnik/8242295/webrev.00/ >>> <http://cr.openjdk.java.net/~lmesnik/8242295/webrev.00/> >>> bug: https://bugs.openjdk.java.net/browse/JDK-8242295 >>> <https://bugs.openjdk.java.net/browse/JDK-8242295>
