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

stack commented on HBASE-795:
-----------------------------

This is a nice patch.  Applies fine.  I took it for a spin doing the 
prescription outlined above.  All works nicely.   Only issues I see is what if 
there exists already a table named 'tables' and the lack of documentation of 
REST in general (not your fault).  

Maybe you should rename the 'tables' switch to be a name you know is illegal as 
a legit tablename: e.g. _tables or some such (See 
HTableDescriptor.isLegalTableName for rules).

Meantime, I'm going to apply this since it adds nice functionality.  If you get 
a chance, add package doc. to REST, if only to point back at the wiki page, and 
fix the 'tables' issue if you come up w/ an idea better than mine above.

> More Table operation in TableHandler for REST interface
> -------------------------------------------------------
>
>                 Key: HBASE-795
>                 URL: https://issues.apache.org/jira/browse/HBASE-795
>             Project: Hadoop HBase
>          Issue Type: New Feature
>          Components: rest
>            Reporter: sishen.freecity
>         Attachments: 795.trunk.rest_table.patch
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> In the current implementation, only the metadata query of table allowed. It's 
> not convinent to use the REST interface to write a third-party client 
> library. I think the functionality of REST interface should be similar with 
> the hbase shell. So user can create/show/update/delate/enable/disable the 
> table.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to