Looks good. Nit: adding a space between catch and ( would be good.
Mandy
On 8/13/2014 5:43 AM, Daniel Fuchs wrote:
Please find below a fix for
8052403: java/util/logging/CheckZombieLockTest.java fails
with NoSuchFileException
https://bugs.openjdk.java.net/browse/JDK-8052403
The issue here is that CheckZombieLockTest and CheckLockLocationTest use
(create and delete) the same writable-dir under /tmp.
When jtreg is run with a high concurrency factor this sometimes results
in one of the tests deleting the directory that the other is
running. I could verify this by triggering the same failure with:
jtreg -verbose:all -jdk $MYJDK -conc:12 \
jdk/test/java/util/logging/Check*
The fix simply arranges for CheckZombieLockTest to use a different
directory than that used by CheckLockLocationTest (and also improves
the diagnostic by arranging for setUp() to fail fast if the test
lock file cannot be created).
Webrev:
http://cr.openjdk.java.net/~dfuchs/webrev_8052403/webrev.00/
best regards,
-- daniel
