accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Havanki (JIRA)" <j...@apache.org>
Subject [jira] [Assigned] (ACCUMULO-2615) Simplify server configuration caching and ZK watchers
Date Wed, 02 Apr 2014 17:15:18 GMT

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

Bill Havanki reassigned ACCUMULO-2615:
--------------------------------------

    Assignee: Bill Havanki

> Simplify server configuration caching and ZK watchers
> -----------------------------------------------------
>
>                 Key: ACCUMULO-2615
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2615
>             Project: Accumulo
>          Issue Type: Improvement
>            Reporter: Bill Havanki
>            Assignee: Bill Havanki
>             Fix For: 1.6.1
>
>
> The mechanism used for caching server-related configuration objects and processing ZK
events could do with some simplification. For example:
> * {{NamespaceConfWatcher}} and {{TableConfWatcher}} objects guess at what they are supposed
to be watching by looking for namespace/table IDs in ZK events; they do not save the IDs they
pertain to.
> * The watcher objects obtain their corresponding configuration objects by reaching through
the {{ServerConfiguration}} cache, instead of just having a reference field.
> * The watcher objects set log4j logging levels for both ZK and o.a.hadoop.io.compress
to WARN, which prohibits debugging when needed. That should be done in log4j properties/XML
instead.
> * Possible bug: {{NamespaceConfWatcher}} expires table configuration observers, not namespace
configuration observers.
> * There is high duplication between the static and instance methods of {{ServerConfiguration}}.
The class serves as both a cache container and a factory. Moreover, despite its name, it isn't
actually a configuration object.
> I may be missing some subtleties of why things are as they are, so please fill me in
as needed.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Mime
View raw message