hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Matt Corgan (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-10801) Ensure DBE interfaces can work with Cell
Date Thu, 20 Mar 2014 06:50:50 GMT

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

Matt Corgan commented on HBASE-10801:
-------------------------------------

I did some tinkering with this a while back, and the approach may be very outdated, but I'll
quickly point it out here.  The DBE's maintain a BufferedDataBlockEncoder.SeekerState which
is a snapshot of a single decoded KV/Cell that is ready to be sent up the read path.

In existing code, it gets deep-copied to a KeyValue: https://github.com/apache/hbase/blob/0.98/hbase-common/src/main/java/org/apache/hadoop/hbase/io/encoding/BufferedDataBlockEncoder.java

But I think SeekerState could be made to implement Cell and therefore we could send a shallow-copied
reference up the read path.  This code is very old and likely has parts that don't make sense
or even compile, but hopefully it still illustrates how SeekerState can extend Cell: https://github.com/hotpads/hbase/blob/cell-seeker-state/hbase-server/src/main/java/org/apache/hadoop/hbase/io/encoding/BufferedDataBlockEncoder.java

I also remember trying two approaches with different strategies regarding when to copy the
primitive fields, and this one was slower, but I don't have the faster one.  Just throwing
that out there in case there's merit to the overall approach.

> Ensure DBE interfaces can work with Cell
> ----------------------------------------
>
>                 Key: HBASE-10801
>                 URL: https://issues.apache.org/jira/browse/HBASE-10801
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: ramkrishna.s.vasudevan
>             Fix For: 0.99.0
>
>
> Some changes to the interfaces may be needed for DBEs or may be the way it works currently
may be need to be modified inorder to make DBEs work with Cells. Suggestions and ideas welcome.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Mime
View raw message