RFR: 8127: Use ProcessHandle API to get PIDs

Christoph Dreis duke at openjdk.org
Sun Oct 15 10:54:20 UTC 2023


On Sun, 15 Oct 2023 10:34:50 GMT, Christoph Langer <clanger at openjdk.org> wrote:

>> Hi,
>> 
>> this PR uses `ProcessHandle.current().pid()` that was introduced in Java 9 rather than parsing the output of `ManagementFactory.getRuntimeMXBean().getName()`.
>> 
>> If you think this is worthwhile, I'd appreciate a ticket and sponsoring.
>> 
>> Cheers,
>> Christoph
>
> @dreis2211, please enable GHA testing to avoid such issues like the checkstyle problems introduced with #518. Thanks.

@RealCLanger thanks for the review. Is there a problem with this PR? What do you mean by enabling GHA testing? Aren't the Github Actions executed here?

-------------

PR Comment: https://git.openjdk.org/jmc/pull/522#issuecomment-1763351879


More information about the jmc-dev mailing list