hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jean-Daniel Cryans (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-7826) Improve Hbase Thrift v1 to return results in sorted order
Date Thu, 11 Jul 2013 23:53:48 GMT

    [ 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

Mime
View raw message