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 2014/03/31 03:52:15 UTC

[jira] [Commented] (ACCUMULO-2592) Add new interfaces that better allow extension/implementations of Accumulo clusters

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

Christopher Tubbs commented on ACCUMULO-2592:
---------------------------------------------

That particular experimental annotation was specifically introduced for the configuration. If we need something more broadly than that, I'd prefer we design that in as a public API annotation, not reuse something that was meant for being internal use only. I'd also prefer that we discuss it and explicitly make a decision about how we want to communicate intent to API consumers, so we can document it, rather than simply use something internally and have API consumers make assumptions about what intentions we meant with the use of a particular annotation.

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