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

Matt Warhaftig commented on HBASE-15870:
----------------------------------------

[~yuzhih...@gmail.com] would you be available to do a code review on this 
patch?  Sorry to point at you for a request but I like your work. :-)

As far as:
{quote}
"You might want to consider changing the URL structure a bit to be of the form 
'columns=...&row=...&row...' "
{quote}
My opinion was that including the 'columns' property on the query string could 
create confusion and add an edge case of multiple 'columns' tags.  Also, having 
columns as a path item matches existing REST urls (ex. GET on a single cell 
value).  Still, let us see the code reviewer's opinion.

> Specify columns in REST multi gets
> ----------------------------------
>
>                 Key: HBASE-15870
>                 URL: https://issues.apache.org/jira/browse/HBASE-15870
>             Project: HBase
>          Issue Type: New Feature
>          Components: REST
>            Reporter: Dean Gurvitz
>            Assignee: Matt Warhaftig
>            Priority: Minor
>         Attachments: hbase-15870-v1.patch
>
>
> The REST multi-gets feature currently does not allow specifying only certain 
> columns or column families. Adding support for these should be quite simple 
> and improve the usability of the multi-gets feature.



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

Reply via email to