[
https://issues.apache.org/jira/browse/ACCUMULO-2592?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13954956#comment-13954956
]
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: 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 [email protected], 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)
