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