Hi Ujwal,

Could you please add a test-case to validate your changes? You can spawn a new process and it can exchange its pid to its parent via System.out/in or via sockets.

Also, VMManagementImpl:145, the change from getProcessId to getVmPid seems unnecessary.

-Harsha


On Tuesday 10 October 2017 05:17 PM, Ujwal Vangapally wrote:
Kindly review the changes made.

https://bugs.openjdk.java.net/browse/JDK-8044122

webrev : http://cr.openjdk.java.net/~uvangapally/webrev/2017/8044122/webrev.00/

CSR : https://bugs.openjdk.java.net/browse/JDK-8189091

Thanks,

Ujwal.


Reply via email to