hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Enis Soztutar (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-11797) Create Table interface to replace HTableInterface
Date Tue, 26 Aug 2014 23:54:00 GMT

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

Enis Soztutar commented on HBASE-11797:
---------------------------------------

Do you want to pick up the HTable changes in https://issues.apache.org/jira/secure/attachment/12641764/hbase-10602_v1.patch
as well or you want to do it in the Connection issue? 

Can you please add: 
{code}
+ * @deprecated use {@link Table} instead
{code} to HTableInterface.java. 

Lets keep this signature in HTI only, and add a existsAll() to Table which returns boolean[],
wdyt? 
{code}
+  Boolean[] exists(List<Get> gets) throws IOException;
{code}

why the changes in test code? Tests failing because of a different issue? 
I'll open another issue to remove HTableInterface, etc from master (2.0 line). 

> Create Table interface to replace HTableInterface
> -------------------------------------------------
>
>                 Key: HBASE-11797
>                 URL: https://issues.apache.org/jira/browse/HBASE-11797
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Carter
>            Assignee: Carter
>         Attachments: HBASE_11797.patch, HBASE_11797_v2.patch
>
>
> Basically doing this:
> {code}
> interface Table {
>   // get, put related stuff
> }
> @Deprecated
> interface HTableInterface extends Table {
>   // users are encouraged to use the new Table interface
> }
> class HTable extends Table {
>   // all HTable constructors are deprecated
>   // Users are not encouraged to see this class
> }
> {code}
> I'm proposing that in this JIRA I move everything from HTableInterface to Table except
the following:
> * Anything deprecated
> * Anything @InterfaceAudience.Private ({{coprocessorService(...)}} and {{batchCoprocessorService(...)}})



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

Mime
View raw message