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

Andrew Purtell commented on HBASE-12991:
----------------------------------------

+1 for the REST changes

This:
{code}
diff --git 
hbase-server/src/main/java/org/apache/hadoop/hbase/util/ConnectionCache.java 
hbase-server/src/main/java/org/apache/hadoop/hbase/util/ConnectionCache.java
index 21714af..8bcdda9 100644
--- hbase-server/src/main/java/org/apache/hadoop/hbase/util/ConnectionCache.java
+++ hbase-server/src/main/java/org/apache/hadoop/hbase/util/ConnectionCache.java
[...]
{code}

and this:
{code}
diff --git 
hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
 
hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
index 9f23c09..2752a1a 100644
--- 
hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
+++ 
hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
[...]
{code}

are out of scope per the issue description. Or we can make this about replacing 
HBaseAdmin with Admin more generally. Perhaps sweep up more occurrences if 
there are any. 

Anyway, I'm happy to commit the patch and resolve the issue as is minus the 
called out changes.

> Use HBase 1.0 interfaces in hbase-rest
> --------------------------------------
>
>                 Key: HBASE-12991
>                 URL: https://issues.apache.org/jira/browse/HBASE-12991
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Solomon Duskis
>            Assignee: Solomon Duskis
>             Fix For: 2.0.0, 1.0.1
>
>         Attachments: HBASE-12991.patch
>
>
> hbase-rest uses HTable and HBaseAdmin under the covers.  They should use the 
> new hbase 1.0 interfaces instead.



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

Reply via email to