RFR: 8259070: Add jcmd option to dump CDS [v3]

Yumin Qi minqi at openjdk.java.net
Thu Mar 18 03:34:58 UTC 2021


On Wed, 10 Mar 2021 04:28:04 GMT, Ioi Lam <iklam at openjdk.org> wrote:

>> Yumin Qi has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix white space in CDS.java
>
> src/java.base/share/classes/jdk/internal/misc/CDS.java line 256:
> 
>> 254: 
>> 255:             // Do not take parent env which will cause dumping fail.
>> 256:             Process proc = Runtime.getRuntime().exec(cmds.toArray(new String[0]),
> 
> Could you explain why the parent's env variables will cause dumping to fail?

I found jtreg env will be brought in to the children env which is not needed in this case. Add comment.

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

PR: https://git.openjdk.java.net/jdk/pull/2737


More information about the hotspot-runtime-dev mailing list