[jira] [Commented] (ACCUMULO-3604) connector.instanceOperations().setSystemProperty(String,String) returns silently for non-changeable ZK Property's

["Christopher Tubbs (JIRA)" <ji...@apache.org>]

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

Christopher Tubbs commented on ACCUMULO-3604:
---------------------------------------------

I was just thinking that the main point of it was to inform the client, for them to deal with, and that's expected for client-side code.

Your description of server-side code behaving as a client... yeah, we might not see this here, but it'd get passed up to somewhere where eventually we'd see it in the logs. I don't really expect server-side code to be executing this client code at all... but if, for some reason it does, it seems like throwing it back up to the server-side code calling it would be a reasonable thing to do.

I don't object to the log statement. I just thought it'd be worth questioning whether it had value. Either way is fine with me.

> connector.instanceOperations().setSystemProperty(String,String) returns silently for non-changeable ZK Property's
> -----------------------------------------------------------------------------------------------------------------
>
>                 Key: ACCUMULO-3604
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-3604
>             Project: Accumulo
>          Issue Type: Bug
>          Components: master
>    Affects Versions: 1.5.0, 1.5.1, 1.5.2, 1.6.0, 1.6.1, 1.6.2
>            Reporter: Josh Elser
>            Assignee: Jeffrey S Schwartz
>              Labels: newbie, summit2015
>             Fix For: 1.5.3, 1.6.3, 1.8.0, 1.7.1
>
>         Attachments: ACCUMULO-3604.patch
>
>
> Only a subset of the configuration {{Property}}'s in Accumulo are modifiable via ZooKeeper (defined by {{Property.isValidZooProperty}}).
> {{connector.instanceOperations().setProperty(String,String)}} updates the provided property name with the given value in ZooKeeper. The thing that is never mentioned is that only a subset of the properties in Accumulo are allowed to be overriden in ZooKeeper. Furthermore, the user receives no indication that their call failed.
> The Javadoc on {{setSystemProperty(String,String)}} should be updated to inform the users that only some properties can be changed by this method, and some information should be returned back to the user to let them know that their call did not succeed (likely an Exception).



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