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

Hudson commented on HBASE-23825:
--------------------------------

FAILURE: Integrated in Jenkins build HBase-1.3-IT #676 (See 
[https://builds.apache.org/job/HBase-1.3-IT/676/])
HBASE-23825 Increment proto conversion is broken (achouhan: rev 
666c78d5ed03d30493d75c7f322d28e1a18601aa)
* (edit) 
hbase-protocol/src/main/java/com/google/protobuf/HBaseZeroCopyByteString.java
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/protobuf/TestProtobufUtil.java


> Increment proto conversion is broken
> ------------------------------------
>
>                 Key: HBASE-23825
>                 URL: https://issues.apache.org/jira/browse/HBASE-23825
>             Project: HBase
>          Issue Type: Bug
>          Components: Increment
>    Affects Versions: 1.4.0, 1.2.6, 1.3.2, 1.4.1, 1.5.0, 1.1.11, 1.3.3, 1.4.2, 
> 1.4.3, 1.4.4, 1.4.5, 1.3.2.1, 1.4.6, 1.4.8, 1.4.7, 1.4.9, 1.4.10, 1.3.4, 
> 1.3.5, 1.3.6, 1.4.11, 1.4.12
>            Reporter: Abhishek Singh Chouhan
>            Assignee: Abhishek Singh Chouhan
>            Priority: Blocker
>             Fix For: 1.6.0, 1.3.7, 1.4.13
>
>
> While converting the request back to Increment using ProtobufUtil.toIncrement 
> we incorrectly use the optimization to avoid copying the byte 
> array(HBaseZeroCopyByteString#zeroCopyGetBytes) on a BoundedByteString. The 
> optimization was only meant for LiteralByteString where it is safe to use the 
> backing byte array, however it ends up being used to BoundedByteString which 
> is a subclass of LiteralByteString. This essentially breaks increments since 
> we end up creating wrong cells on the server side. 



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to