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 serviceability-dev mailing list