RFR: 8290313: Produce warning when user specified java.io.tmpdir directory doesn't exist [v4]
Sean Coffey
coffeys at openjdk.org
Sat Oct 15 09:50:11 UTC 2022
On Fri, 14 Oct 2022 19:25:54 GMT, Weibing Xiao <duke at openjdk.org> wrote:
>> 8290313: Produce warning when user specified java.io.tmpdir directory doesn't exist
>
> Weibing Xiao has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 11 commits:
>
> - Merge branch 'master' into improve-directory-not-existing-error-message
> - new approach: print warning in StaticProperty.java
> - Merge branch 'master' into improve-directory-not-existing-error-message
> - add the change for nio and update the code according to the comments
> - new approach
> - change based on the review
> - updating according to the comments
> - Merge branch 'master' of https://github.com/openjdk/jdk into improve-directory-not-existing-error-message
> - update error message
> - error message for non-existing directory
> - ... and 1 more: https://git.openjdk.org/jdk/compare/8487c56f...2da71bdc
Seems like there's a difference of opinion on when such a configuration issue should be reported. Reporting at start up will cost cycles (esp since there's an IO stat call) Maybe we can revert the patch to the original style which is to report the issue at time of tmpFile creation failure.
The original intent of the bug is to improve the exception message reported to end user for the extremely rare case of when java.io.tmpdir contains a bad value. The current exception is something like:
java.io.IOException: No such file or directory
at java.io.UnixFileSystem.createFileExclusively(Native Method)
The exception message needs to suggest that the java.io.tmpdir property is a bad one. (no need to even print the directory value IMO)
-------------
PR: https://git.openjdk.org/jdk/pull/9989
More information about the core-libs-dev
mailing list