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

Aleksandr Shulman commented on HBASE-8345:
------------------------------------------

Sounds good. I should have put the link to the review in the JIRA. I'll address 
those issues here, but let's take the conversion to the code review

Patch -- Intended for 0.94 only.
Parsing -- I'll look into this.
Naming -- I felt it was a more useful and descriptive name than 'sb'. 
                
> Add all available resources in o.a.h.h.rest.RootResource and VersionResource 
> to o.a.h.h.rest.client.RemoteAdmin
> ---------------------------------------------------------------------------------------------------------------
>
>                 Key: HBASE-8345
>                 URL: https://issues.apache.org/jira/browse/HBASE-8345
>             Project: HBase
>          Issue Type: Improvement
>          Components: Client, REST
>    Affects Versions: 0.94.6.1
>            Reporter: Aleksandr Shulman
>            Assignee: Aleksandr Shulman
>            Priority: Minor
>              Labels: rest_api
>             Fix For: 0.94.8
>
>         Attachments: HBASE-8345-v1.patch
>
>
> In our built-in REST clients, we should add in more of the available REST 
> resources. This will allow more thorough testing of the REST API, 
> particularly with IntegrationTest.
> These clients are located in the o.a.h.h.rest.client package.
> In this case, I want to add the resources not already included in / and 
> /version to o.a.h.h.rest.client.RemoteAdmin. This includes, /status/cluster, 
> /version/rest and /version/cluster, among others.
> The RemoteAdmin class is a logical place for these methods because it is not 
> related to a specific table (those methods should go into RemoteHTable).

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to