On Thu, 11 Jul 2024 14:51:18 GMT, Kevin Walls <kev...@openjdk.org> wrote:
> Follow-on from JDK-8207908. > > Two tests are broken by that change: > sun/management/jmxremote/startstop/JMXStartStopTest.java > sun/management/jmxremote/startstop/JMXStatusPerfCountersTest.java > > These are additional tests which use jcmd and an application name pattern to > find a process. > They should use the PID to avoid finding a process from some other concurrent > test invocation. > So it's good to have the same kind of change applied here too. test/jdk/sun/management/jmxremote/startstop/JMXStartStopTest.java line 351: > 349: pid = p.pid(); > 350: jcmd = new ManagementAgentJcmd(p, verbose); > 351: I don't think you want a blank line here. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/20138#discussion_r1674217138
