Okay, thanks!
Serguei

On 4/6/20 18:54, Chris Plummer wrote:
It's not a test. It a subclassed LingeredApp with code added to print all the system properties. But in any case I'm going to remove the problem listing in this webrev since I will push https://bugs.openjdk.java.net/browse/JDK-8242235 first, which gets rid of the need to problem list on solaris.

Chris

On 4/6/20 6:31 PM, [email protected] wrote:
Hi Chris,

Quick question:
Should theserviceability/sa/LingeredAppSysProps.java be also problem listed?

Thanks,
Serguei


On 4/5/20 22:49, Chris Plummer wrote:
Hello,

Please help review the following:

https://bugs.openjdk.java.net/browse/JDK-8242165
http://cr.openjdk.java.net/~cjplummer/8242165/webrev.03

Please see the CR for an explanation of the bug and the fix. If you need some help with the SA code, let me know and I'll provide some details. It was pretty much all new to me, so I tried to put the needed details in the CR.

For the test, I compare the list of properties dumped using 3 different methods to make sure the same set is dumped for each:

1. jhsdb jinfo --sysprops
2. jinfo -sysprops
3. Simple app (LingeredAppSysProps) that calls System.getProperties().list(System.out)

The app output is considered the master list that is compared against the others, and I also make sure each list has the same number of properties.

thanks,

Chris



Reply via email to