RFR: 8127: Use ProcessHandle API to get PIDs [v2]
Christoph Langer
clanger at openjdk.org
Mon Oct 16 06:03:57 UTC 2023
On Sun, 15 Oct 2023 11:29:56 GMT, Christoph Dreis <duke 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
>
> Christoph Dreis has updated the pull request incrementally with one additional commit since the last revision:
>
> 8127: Remove obsolete try/catch in Environment
I think this is not ready yet. As you see, the Pre-submit tests fail (in spotless). I think the spotless issues should be fixed through #525 but I'd like to see it merged and the runs be green here, too.
-------------
PR Comment: https://git.openjdk.org/jmc/pull/522#issuecomment-1763783070
More information about the jmc-dev
mailing list