Anastasia Braginsky commented on HBASE-19930:

[~anoop.hbase] and [~ram_krish], what [~chia7712] wrote is absolutely correct 
and this is the reason for the fix. Indeed it sounds odd that ImmutableMSLAB is 
updated, but this is what need to happen in this very special case. Gali will 
add the comment. But after that I'll commit since enough people have taken a 
look on the fix.

> fix ImmutableMemStoreLAB#forceCopyOfBigCellInto
> -----------------------------------------------
>                 Key: HBASE-19930
>                 URL: https://issues.apache.org/jira/browse/HBASE-19930
>             Project: HBase
>          Issue Type: Bug
>    Affects Versions: 2.0.0-beta-1
>            Reporter: Gali Sheffi
>            Assignee: Gali Sheffi
>            Priority: Major
>         Attachments: HBASE-19930-V01.patch, HBASE-19930-V02.patch, 
> HBASE-19930-V03.patch, HBASE-19930-V04.patch
> This issue is about fixing ImmutableMemStoreLAB#forceCopyOfBigCellInto.
> Following a comment in HBASE-19133 regarding a bug in 
> ImmutableMemStoreLAB#forceCopyOfBigCellInto (assuming this method is never 
> called for an ImmutableMemStoreLAB, and just throwing an 
> IllegalStateException whenever called), the forceCopyOfBigCellInto method now 
> performs the copy of big cells on the first MSLABImpl in its mslabs 
> linked-list.

This message was sent by Atlassian JIRA

Reply via email to