accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Josh Elser (JIRA)" <>
Subject [jira] [Commented] (ACCUMULO-2143) Add convenience method to get ZooKeeper access with MiniAccumuloCluster
Date Mon, 24 Mar 2014 17:08:45 GMT


Josh Elser commented on ACCUMULO-2143:

bq. The state in ZooKeeper, for that matter, doesn't seem like a thing clients should need
to be verifying. But maybe I'm mistaken in thinking of the MAC as a tool for black box testing?

Personally, I would disagree. The thing I like most about MAC is that it can be a full-fledged
Accumulo testing framework (as honest as MiniDFSCluster is, at least) *in addition* to something
that provides an accurate testing harness for users.

bq. That's pretty easy, actually. ZooKeeper zk = new ZooKeeper(connectString, timeout, null);

If we would say "just make your own ZooKeeper object", you might miss out on the semantics
of the {{instance.zookeeper.timeout}}. I can't really think of a reason (in a normal situation)
where that would be a loss, so I guess this is really a syntactic-sugar sort of ticket. 

That said, if the general consensus is that adding such a method would be superfluous, I'm
ok with it. [~mjwall], if you've already done this, please say so. I definitely don't want
to look a gift-horse in the mouth (if you've already made the changes, I'd rather accept and
apply them).

> Add convenience method to get ZooKeeper access with MiniAccumuloCluster
> -----------------------------------------------------------------------
>                 Key: ACCUMULO-2143
>                 URL:
>             Project: Accumulo
>          Issue Type: Improvement
>          Components: mini
>            Reporter: Josh Elser
>            Assignee: Michael Wall
>              Labels: newbie
>             Fix For: 1.7.0
> When using MAC, it may be convenient to get a ZooKeeper object (or a ZooReaderWriter).
> The user has the ability to figure out or set all of the information they need to make
their own connection, but it would be nice to just have a {{MiniAccumuloCluster.getZooKeeper()}}

This message was sent by Atlassian JIRA

View raw message