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

Lars Hofhansl commented on HBASE-11805:
---------------------------------------

Looks like this potentially breaks Phoenix (and maybe other things) with 
exceptions like these:
{code}
java.lang.NullPointerException
        at org.apache.hadoop.hbase.util.Bytes.toShort(Bytes.java:845)
        at org.apache.hadoop.hbase.util.Bytes.toShort(Bytes.java:832)
        at org.apache.hadoop.hbase.KeyValue.getRowLength(KeyValue.java:1303)
        at org.apache.hadoop.hbase.KeyValue.getFamilyOffset(KeyValue.java:1319)
        at org.apache.hadoop.hbase.KeyValue.getFamilyLength(KeyValue.java:1334)
        at org.apache.hadoop.hbase.CellUtil.cloneFamily(CellUtil.java:70)
        at org.apache.hadoop.hbase.KeyValue.getFamily(KeyValue.java:1559)
        at org.apache.hadoop.hbase.replication.regionserver.Replication.scopeWAL
Edits(Replication.java:249)
        at org.apache.hadoop.hbase.replication.regionserver.Replication.visitLog
EntryBeforeWrite(Replication.java:233)
        at 
org.apache.hadoop.hbase.regionserver.wal.FSHLog.doWrite(FSHLog.java:1486)
        at 
org.apache.hadoop.hbase.regionserver.wal.FSHLog.append(FSHLog.java:1023)
        at 
org.apache.hadoop.hbase.regionserver.wal.FSHLog.appendNoSync(FSHLog.java:1054)
        at 
org.apache.hadoop.hbase.regionserver.HRegion.doMiniBatchMutation(HRegion.java:2553)
{code}


> KeyValue to Cell Convert in WALEdit APIs
> ----------------------------------------
>
>                 Key: HBASE-11805
>                 URL: https://issues.apache.org/jira/browse/HBASE-11805
>             Project: HBase
>          Issue Type: Improvement
>          Components: wal
>            Reporter: Anoop Sam John
>            Assignee: Anoop Sam John
>             Fix For: 0.99.0, 2.0.0, 0.98.7
>
>         Attachments: HBASE-11805.patch, HBASE-11805_0.98.patch, 
> HBASE-11805_0.98_V2.patch, HBASE-11805_0.99.patch, HBASE-11805_V2.patch, 
> HBASE-11805_V3.patch
>
>
> In almost all other main interface class/APIs we have changed KeyValue to 
> Cell. But missing in WALEdit. This is public marked for Replication (Well it 
> should be for CP also) 
> These 2 APIs deal with KVs
> add(KeyValue kv)
> ArrayList<KeyValue> getKeyValues()
> Suggest deprecate them and add for 0.98
> add(Cell kv) 
> List<Cell> getCells()
> And just replace from 1.0



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

Reply via email to