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

Craig Muchinsky commented on HBASE-7621:
----------------------------------------

I like the idea of using either hex escaping or URL encoding for simplicity, 
however based on my testing the hex escaping by itself isn't URL friendly, so 
it had to be URL encoded. I haven't tried just using URL encoding by itself. 
One argument for keeping both would be consistency with the shell, which 
already supports hex notation.
                
> REST server doesn't support binary row keys
> -------------------------------------------
>
>                 Key: HBASE-7621
>                 URL: https://issues.apache.org/jira/browse/HBASE-7621
>             Project: HBase
>          Issue Type: Bug
>          Components: REST
>    Affects Versions: 0.94.0, 0.96.0
>            Reporter: Craig Muchinsky
>            Assignee: Andrew Purtell
>
> The REST server doesn't seem to support using binary (MD5 for example) row 
> keys. I believe the root cause of this is the use of Bytes.toBytes() in the 
> RowSpec.parseRowKeys() method. Based on the use of Bytes.toStringBinary() 
> within RemoteHTable.buildRowSpec(), I believe the converse function 
> Bytes.toBytesBinary() should be used for row key parsing in 
> RowSpec.parseRowKeys().
> I also noticed that the RemoteHTable.buildRowSpec() method isn't URL encoding 
> the row key, which is a mismatch to the logic in RowSpec.parseRowKeys() which 
> performs URL decoding for both the start and stop row keys.

--
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