accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christopher Tubbs (JIRA)" <>
Subject [jira] [Commented] (ACCUMULO-2592) Add new interfaces that better allow extension/implementations of Accumulo clusters
Date Mon, 31 Mar 2014 05:49:14 GMT


Christopher Tubbs commented on ACCUMULO-2592:

I think it's probably a little late for 1.6.0 to begin discussing/documenting what we think
"experimental" should mean in order to start annotating the API with that. What exactly does
it mean to be experimental? The scope for the configuration annotations was very narrow and
functional (as implementation detail to limit exposure of properties in the documentation
and shell for features still in some stage of development). But, if we start using it more
broadly, we're going to have to define the semantics of that annotation.

Perhaps just a comment in the javadoc would be sufficient (which was my suggestion on the
mailing list thread)? I'd prefer not to add more Hadoop dependencies just to borrow their
annotations, and I'd also rather not blindly borrow the semantics of their annotations without
at least some mailing list discussion to see if they are in line with our API goals. I'll
create a sub-task for ACCUMULO-2589, to create annotations with well-defined semantics for
the API, but for this, I think a simple javadoc comment is sufficient..

> Add new interfaces that better allow extension/implementations of Accumulo clusters
> -----------------------------------------------------------------------------------
>                 Key: ACCUMULO-2592
>                 URL:
>             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

View raw message