[jira] [Commented] (ACCUMULO-3548) Classes leaked through InstanceOperations into public API methods

["Josh Elser (JIRA)" <ji...@apache.org>]

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

Josh Elser commented on ACCUMULO-3548:
--------------------------------------

Another option is to advertise that these objects are subject to change, but I feel like that's an insufficient solution because there isn't no "recommended stable" way to do them. These methods were added for user consumption, we should make sure that they're actually stable.

> Classes leaked through InstanceOperations into public API methods
> -----------------------------------------------------------------
>
>                 Key: ACCUMULO-3548
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-3548
>             Project: Accumulo
>          Issue Type: Bug
>          Components: client
>            Reporter: Josh Elser
>            Priority: Critical
>             Fix For: 1.7.0
>
>
> Looking through ACCUMULO-3204, I noticed that we're (inadvertently?) using a few classes in the public API without having them defined in the public API.
> {{connector.instanceOperations().getActiveScans()}} is what uses ActiveScan in the public API. This provides a way for users to be relying on something that should be stable that doesn't hold us to providing stability. ActiveScan also leaks Column, KeyExtent, ScanType and ScanState.
> * {{activeScan.getColumns()}}
> * {{activeScan.getExtent()}}
> * {{activeScan.getType()}}
> * {{activeScan.getState()}}
> It looks like {{ActiveCompaction}} does similar things. {{TimeType}} is also used {{NewTableConfiguration}}.
> We should make sure these classes are in the public API (force us to ensure they stay stable).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)