accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Josh Elser (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-4026) Setting ACCUMULO_CLIENT_CONF_PATH with no client.conf file present throws ConfigurationException
Date Wed, 14 Oct 2015 15:21:06 GMT

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

Josh Elser commented on ACCUMULO-4026:
--------------------------------------

bq. I'm not sure this property is supposed to point to a file

Speaking from how the code was written (and that there's no attempt to append client.conf
to the end of the value), it seems rather apparent to me that it was meant to point to a file.
That is of course just how the code looks now which isn't guaranteed what the original intent
was :)

> Setting ACCUMULO_CLIENT_CONF_PATH with no client.conf file present throws ConfigurationException
> ------------------------------------------------------------------------------------------------
>
>                 Key: ACCUMULO-4026
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-4026
>             Project: Accumulo
>          Issue Type: Bug
>          Components: client, core
>    Affects Versions: 1.7.0
>            Reporter: Jim Klucar
>            Assignee: Jim Klucar
>            Priority: Trivial
>
> Setting the environment variable ACCUMULO_CLIENT_CONF_PATH to a directory that exists
but does not contain a file named client.conf throws
> org.apache.commons.configuration.ConfigurationException: Cannot load a configuration
from a directory
> To demo this, simply set ACCUMULO_CLIENT_CONF_PATH to /tmp, make sure client.conf doesn't
exist in /tmp and try to run the accumulo shell.
> The problem is ClientConfiguration.java on line 221 checks File.canRead() which is true
for a Directory and a File. The solution is to make the branch check File.isFile() &&
File.canRead()



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

Mime
View raw message