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