[ https://issues.apache.org/jira/browse/HBASE-23825?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17033988#comment-17033988 ]
Abhishek Singh Chouhan edited comment on HBASE-23825 at 2/10/20 10:16 PM: -------------------------------------------------------------------------- This is not a problem in master and 2.x since we reverted HBASE-18026 from those branches. FYI [~andrew.purt...@gmail.com] was (Author: abhishek.chouhan): This is not a problem in master and 2.x since we reverted HBASE-18026. FYI [~andrew.purt...@gmail.com] > Increment proto conversion is broken > ------------------------------------ > > Key: HBASE-23825 > URL: https://issues.apache.org/jira/browse/HBASE-23825 > Project: HBase > Issue Type: Bug > Affects Versions: 1.5.0, 1.3.6, 1.4.12 > Reporter: Abhishek Singh Chouhan > Assignee: Abhishek Singh Chouhan > Priority: Major > > 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)