You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "Christopher Tubbs (JIRA)" <ji...@apache.org> on 2015/05/21 23:22:17 UTC

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

     [ https://issues.apache.org/jira/browse/ACCUMULO-3604?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Christopher Tubbs updated ACCUMULO-3604:
----------------------------------------
    Fix Version/s:     (was: 1.7.1)
                       (was: 1.6.3)
                       (was: 1.5.3)

Bumping to 1.8.0 only, since it represents a change in behavior of public API, and it's not entirely clear to me that this is a bug (the behavior is undefined).

If we want to apply to an earlier version, we can revisit the discussion about which branch to apply it to.

Also, [~jschwartz73], I believe there were a few comments above that were left unresponded-to, which has held up this patch from getting applied. If you wouldn't mind taking a look at those, I'd appreciate it. If you don't have time, and somebody else wishes to pick up where you left off, that seems fine, too.

> 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.8.0
>
>         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)