Hi Roger,
Running in othervm looks good to me.
The only thing I wonder about is these lines which are
removed:
79 // Log remaining processes
80 ProcessBuilder pb = new ProcessBuilder("/bin/ps", "-ef");
81 pb.inheritIO();
82 Process p2 = pb.start();
83 p2.waitFor();
Isn't that removing diagnostic information?
best regards,
-- daniel
On 02/08/16 14:45, Roger Riggs wrote:
Please review a test fix for an intermittently failing ProcessBuilder test.
The test should be run in a separate vm so it only sees Zombies from
processes it creates.
When run in-process with jtreg, it may see Zombies from other processes
created by jtreg.
Additional diagnostic information is added as well in case of recurrence.
The test is moved back to tier1.
Webrev:
http://cr.openjdk.java.net/~rriggs/webrev-zombies-8160151/
Issue:
https://bugs.openjdk.java.net/browse/JDK-8160151
Thanks, Roger
