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

Heng Chen commented on HBASE-15714:
-----------------------------------

If getRowLock called in doMiniBatchMutation,  it means it passed 
checkBatchOp.checkRow,  otherwise current operation will be skipped.
So i think there is no need to do checkRow again in doMiniBatchMutation. 
getRowLock.  

And checkBatchOp only called in doMiniBatchMutation, i think it will be safe 
just skip checkRow in getRowLock in doMiniBatchMutation


> We are calling checkRow() twice in doMiniBatchMutation()
> --------------------------------------------------------
>
>                 Key: HBASE-15714
>                 URL: https://issues.apache.org/jira/browse/HBASE-15714
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Enis Soztutar
>             Fix For: 2.0.0, 1.4.0
>
>         Attachments: HBASE-15714.patch
>
>
> In {{multi()}} -> {{doMiniBatchMutation()}} code path, we end up calling 
> {{checkRow()}} twice, once from {{checkBatchOp()}} and once from 
> {{getRowLock()}}. 
> See [~anoop.hbase]'s comments at 
> https://issues.apache.org/jira/browse/HBASE-15600?focusedCommentId=15257636&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15257636.
>  



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

Reply via email to