jmx-dev RFR: 8336257: Additional tests in jmxremote/startstop to match on PID not app name [v2]
Kevin Walls
kevinw at openjdk.org
Thu Jul 11 15:38:30 UTC 2024
On Thu, 11 Jul 2024 15:22:54 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:
>> Kevin Walls has updated the pull request incrementally with one additional commit since the last revision:
>>
>> line
>
> 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.
I don't mind the additional space, but ok have removed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20138#discussion_r1674239544
More information about the jmx-dev
mailing list