RFR: 8252883: AccessDeniedException caused by delayed file deletion on Windows [v4]

Daniel Fuchs dfuchs at openjdk.java.net
Wed Feb 17 12:28:53 UTC 2021


On Wed, 17 Feb 2021 11:50:05 GMT, Evan Whelan <ewhelan at openjdk.org> wrote:

>> Hi,
>> 
>> Please review this fix for JDK-8252883. This handles the case when an AccessDeniedException is being thrown on Windows, due to a delay in deleting the lock file it is trying to write to.
>> 
>> This fix passes all testing.
>> 
>> Kind regards,
>> Evan
>
> Evan Whelan has updated the pull request incrementally with one additional commit since the last revision:
> 
>   8252883: Remove ClassPathException copyright statement

test/jdk/java/util/logging/FileHandlerAccessTest.java line 45:

> 43:         if (!(args.length == 2 || args.length == 1)) {
> 44:             System.out.println("Usage error: expects java FileHandlerAccessTest [process/thread] <count>");
> 45:             return;

Ah - sorry - since this is a test, instead of return you should probably throw an exception - e.g.:
throw new IllegalArgumentException("Usage error: expects java FileHandlerAccessTest [process/thread] <count>");

test/jdk/java/util/logging/FileHandlerAccessTest.java line 47:

> 45:             return;
> 46:         }
> 47:         else if (args.length == 2) {

nit: `} else if (...) {`

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

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


More information about the core-libs-dev mailing list