Mailing-List: contact notifications-help@accumulo.apache.org; run by ezmlm
Precedence: bulk
Reply-To: jira@apache.org
Date: Mon, 24 Mar 2014 17:08:45 +0000 (UTC)
From: "Josh Elser (JIRA)" <jira@apache.org>
To: notifications@accumulo.apache.org
Message-ID: <JIRA.12687582.1389061984894.132563.1395680925682@arcas>
In-Reply-To: <JIRA.12687582.1389061984894@arcas>
References: <JIRA.12687582.1389061984894@arcas>
Subject: [jira] [Commented] (ACCUMULO-2143) Add convenience method to get
 ZooKeeper access with MiniAccumuloCluster
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

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: https://issues.apache.org/jira/browse/ACCUMULO-2143
>             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()}} method.


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