[
https://issues.apache.org/jira/browse/HBASE-7826?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13706483#comment-13706483
]
Jean-Daniel Cryans commented on HBASE-7826:
-------------------------------------------
Yeah there's a mixup in the patch, the rowResultFromHBase method that takes a
single results doesn't have a boolean.
The new patch still have a few offending lines:
{noformat}
+ for(KeyValue kv : result_.raw()) {
+ protected synchronized int addScanner(ResultScanner scanner,boolean
sortColumns) {
{noformat}
And this should be on the same line:
{noformat}
+ }
+ else {
{noformat}
So those plus Ted's comment about rowResultFromHBase should put us close to
being able to commit. The other thing missing is a patch for trunk.
> Improve Hbase Thrift v1 to return results in sorted order
> ---------------------------------------------------------
>
> Key: HBASE-7826
> URL: https://issues.apache.org/jira/browse/HBASE-7826
> Project: HBase
> Issue Type: New Feature
> Components: Thrift
> Affects Versions: 0.94.0
> Reporter: Shivendra Pratap Singh
> Assignee: Shivendra Pratap Singh
> Priority: Minor
> Labels: Hbase, Thrift
> Attachments: hbase_7826.patch, hbase_7826.patch,
> hbase_7826_sortcolumnFlag.1.patch, hbase_7826_sortcolumnFlag.patch
>
>
> Hbase natively stores columns sorted based on the column qualifier. A scan is
> guaranteed to return sorted columns. The Java API works fine but the Thrift
> API is broken. Hbase uses TreeMap that ensures that sort order is maintained.
> However Hbase thrift specification uses a simple Map to store the data. A
> map, since it is unordered doesn't result in columns being returned in a sort
> order that is consistent with their storage in Hbase.
--
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
