Mailing-List: contact notifications-help@accumulo.apache.org; run by ezmlm
Precedence: bulk
Reply-To: jira@apache.org
Date: Mon, 31 Mar 2014 17:27:17 +0000 (UTC)
From: "Josh Elser (JIRA)" <jira@apache.org>
To: notifications@accumulo.apache.org
Message-ID: <JIRA.12705563.1396209029036.37473.1396286837364@arcas>
In-Reply-To: <JIRA.12705563.1396209029036@arcas>
References: <JIRA.12705563.1396209029036@arcas>
Subject: [jira] [Commented] (ACCUMULO-2592) Add new interfaces that better
 allow extension/implementations of Accumulo clusters
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

Josh Elser commented on ACCUMULO-2592:
--------------------------------------

bq. The getDebugPorts() method should probably be dropped from AccumuloCluster, it only makes sense when things are running locally.

Good catch. I was thinking you could do host:port, but that's not the case.

bq.  For a new class the @since tag can be added at the class level and is not needed on the methods.

Will fix.

> Add new interfaces that better allow extension/implementations of Accumulo clusters
> -----------------------------------------------------------------------------------
>
>                 Key: ACCUMULO-2592
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2592
>             Project: Accumulo
>          Issue Type: Improvement
>          Components: mini
>            Reporter: Josh Elser
>            Assignee: Josh Elser
>             Fix For: 1.6.0
>
>         Attachments: 0001-ACCUMULO-2592-Create-AccumuloCluster-and-AccumuloCon-2.patch, 0001-ACCUMULO-2592-Create-AccumuloCluster-and-AccumuloCon.patch
>
>
> From the conversation on dev@a.a.o, this is ticket encapsulate additions to the minicluster codebase. These changes decouple the AccumuloMiniCluster implementation from any "programmatically deployed" Accumulo cluster. 
> It defines an AccumuloConfig and AccumuloCluster, which the existing AccumuloMiniConfig and AccumuloMiniCluster implement with changing the API. These new interfaces and the factory to create implementations of them are marked as experimental and reside in their own package as they are being introduced so late in the 1.6.0 game.
> The existing classes which are in the "public api" are not modified -- the only changes made were done internally. The internal MiniAccumuloCluster and MiniAccumuloConfig classes now implement these new interfaces, have appropriate override annotations, and contain a new {{build}} method on the AccumuloConfiguration implementation to simplify the factory.


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