RFR: 8052403 java/util/logging/CheckZombieLockTest.java fails with NoSuchFileException

Daniel Fuchs daniel.fuchs at oracle.com
Tue Aug 19 13:52:08 UTC 2014

On 8/15/14 10:32 PM, Mandy Chung wrote:
 > Looks good.  Nit: adding a space between catch and ( would be good.

Thanks Mandy.

I took that opportunity to also fix the spaces before and after '+' in
print statements.

For the record the new webrev I'm going to push is here:
(no need to re-review it's mostly noise due to the added white
  spaces now).

best regards,

-- daniel

> 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

More information about the core-libs-dev mailing list