[jira] [Commented] (JCS-163) false error message when reusing a cache directory

["Gary Gregory (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/JCS-163?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15313880#comment-15313880 ] 

Gary Gregory commented on JCS-163:
----------------------------------

Patches (with unit tests) welcome! ;-)

> false error message when reusing a cache directory
> --------------------------------------------------
>
>                 Key: JCS-163
>                 URL: https://issues.apache.org/jira/browse/JCS-163
>             Project: Commons JCS
>          Issue Type: Bug
>          Components: Indexed Disk Cache, JDBC Disk Cache
>    Affects Versions: jcs-2.0-beta-1
>            Reporter: Mark Fulton
>
> JCS logs an {{ERROR}} if the disk cache directory already existed:
> {noformat}
> 63   [main] ERROR org.apache.commons.jcs.auxiliary.disk.AbstractDiskCacheAttributes  - Failed to create directory C:\MyCache
> {noformat}
> This is because [{{AbstractDiskCacheAttributes.setDiskPath(File)}}|http://commons.apache.org/proper/commons-jcs/commons-jcs-core/apidocs/org/apache/commons/jcs/auxiliary/disk/AbstractDiskCacheAttributes.html#setDiskPath(java.io.File)] [uses|http://commons.apache.org/proper/commons-jcs/commons-jcs-core/xref/org/apache/commons/jcs/auxiliary/disk/AbstractDiskCacheAttributes.html#L80] the result of [{{File.mkdirs()}}|http://docs.oracle.com/javase/6/docs/api/java/io/File.html#mkdirs()] as if it returns {{true}} if the directories exist after invocation but this is not the case, it only returns "{{true}} if and only if the directory was created, along with all necessary parent directories; {{false}} otherwise."
> [{{File.exists()}}|http://docs.oracle.com/javase/6/docs/api/java/io/File.html#exists()] (or similar) should be used instead to trigger logging an {{ERROR}} or not.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)