[jira] [Commented] (HBASE-9261) Add cp hooks after {start|close}RegionOperation in batchMutate

["Anoop Sam John (JIRA)" <ji...@apache.org>]

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

Anoop Sam John commented on HBASE-9261:
---------------------------------------

Now when we call postCloseRegionOperation in all cases of close region operation, better we pass the Operation to this hook also.
{code}
+    } catch (IOException e) {
+      lock.readLock().unlock();
+      throw e;
+    }
{code}
You are ensuring the release of the read lock. Better to catch generic Exception in such a case?

> Add cp hooks after {start|close}RegionOperation in batchMutate
> --------------------------------------------------------------
>
>                 Key: HBASE-9261
>                 URL: https://issues.apache.org/jira/browse/HBASE-9261
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: rajeshbabu
>            Assignee: rajeshbabu
>         Attachments: HBASE-9261.patch, HBASE-9261_v2.patch, HBASE-9261_v3.patch
>
>
> These hooks helps for checking Resources(blocking memstore size) and necessary locking on index region while performing batch of mutations. 



--
This message was sent by Atlassian JIRA
(v6.1#6144)