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

2013-12-13 Thread Ted Yu (JIRA)

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

Ted Yu commented on HBASE-9261:
---

How about naming the hook postBatchMutateIndispensably ?

 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, HBASE-9261_v4.patch, HBASE-9261_v5.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.4#6159)


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

2013-12-13 Thread Anoop Sam John (JIRA)

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

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

bq.How about naming the hook postBatchMutateIndispensably ?
+1

 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, HBASE-9261_v4.patch, HBASE-9261_v5.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.4#6159)


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

2013-12-13 Thread Anoop Sam John (JIRA)

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

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

+1

 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, HBASE-9261_v4.patch, HBASE-9261_v5.patch, 
 HBASE-9261_v6.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.4#6159)


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

2013-12-11 Thread rajeshbabu (JIRA)

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

rajeshbabu commented on HBASE-9261:
---

bq. postCompleteBatchMutate - Will it be fine to pass the 
MiniBatchOperationInProgressMutation miniBatchOp also?
It will be better to pass miniBatchOp. Upload new patch for this.

bq.I would say pls have a better name for this API.
I am not getting any other name. Can any one please suggest better name for 
postCompleteBatchMutate hook?
Thanks.

 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, HBASE-9261_v4.patch, HBASE-9261_v5.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.4#6159)


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

2013-12-09 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-9261:
--

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12617820/HBASE-9261_v5.patch
  against trunk revision .

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:green}+1 tests included{color}.  The patch appears to include 6 new 
or modified tests.

{color:green}+1 hadoop1.0{color}.  The patch compiles against the hadoop 
1.0 profile.

{color:green}+1 hadoop1.1{color}.  The patch compiles against the hadoop 
1.1 profile.

{color:green}+1 javadoc{color}.  The javadoc tool did not generate any 
warning messages.

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:red}-1 findbugs{color}.  The patch appears to introduce 2 new 
Findbugs (version 1.3.9) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:green}+1 lineLengths{color}.  The patch does not introduce lines 
longer than 100

{color:red}-1 site{color}.  The patch appears to cause mvn site goal to 
fail.

{color:green}+1 core tests{color}.  The patch passed unit tests in .

Test results: 
https://builds.apache.org/job/PreCommit-HBASE-Build/8099//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/8099//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop2-compat.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/8099//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-prefix-tree.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/8099//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-client.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/8099//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-common.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/8099//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-protocol.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/8099//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-server.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/8099//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-examples.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/8099//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-thrift.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/8099//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop-compat.html
Console output: 
https://builds.apache.org/job/PreCommit-HBASE-Build/8099//console

This message is automatically generated.

 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, HBASE-9261_v4.patch, HBASE-9261_v5.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.4#6159)


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

2013-12-09 Thread rajeshbabu (JIRA)

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

rajeshbabu commented on HBASE-9261:
---

findbugs warnings are not related to this patch.

 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, HBASE-9261_v4.patch, HBASE-9261_v5.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.4#6159)


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

2013-12-09 Thread ramkrishna.s.vasudevan (JIRA)

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

ramkrishna.s.vasudevan commented on HBASE-9261:
---

Looks good. +1

 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, HBASE-9261_v4.patch, HBASE-9261_v5.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.4#6159)


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

2013-12-09 Thread Anoop Sam John (JIRA)

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

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

Sorry I should have given this before
postCompleteBatchMutate  - Will it be fine to pass the 
MiniBatchOperationInProgressMutation miniBatchOp  also?
I would say pls have a better name for this API.



 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, HBASE-9261_v4.patch, HBASE-9261_v5.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.4#6159)


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

2013-12-04 Thread ramkrishna.s.vasudevan (JIRA)

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

ramkrishna.s.vasudevan commented on HBASE-9261:
---

In postCompleteBatchMutate() can we pass a flag to indicate whether it is 
getting called out of success or failure in the finally block?

 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, HBASE-9261_v4.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)


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

2013-12-03 Thread Anoop Sam John (JIRA)

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

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

{code}
+   * Used after startRegionOperation in region operation
+   * @param ctx
+   * @param operation
+   * @throws IOException
+   */
+  void postStartRegionOperation(final 
ObserverContextRegionCoprocessorEnvironment ctx,
{code}
Used after startRegionOperation -  Can you pls better this javadoc?  Same with 
postClose...


 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, HBASE-9261_v4.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)


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

2013-11-21 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-9261:
--

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12615109/HBASE-9261_v4.patch
  against trunk revision .

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:green}+1 tests included{color}.  The patch appears to include 6 new 
or modified tests.

{color:green}+1 hadoop1.0{color}.  The patch compiles against the hadoop 
1.0 profile.

{color:green}+1 hadoop2.0{color}.  The patch compiles against the hadoop 
2.0 profile.

{color:green}+1 javadoc{color}.  The javadoc tool did not generate any 
warning messages.

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:red}-1 findbugs{color}.  The patch appears to introduce 1 new 
Findbugs (version 1.3.9) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:green}+1 lineLengths{color}.  The patch does not introduce lines 
longer than 100

{color:red}-1 site{color}.  The patch appears to cause mvn site goal to 
fail.

 {color:red}-1 core tests{color}.  The patch failed these unit tests:
   org.apache.hadoop.hbase.regionserver.wal.TestLogRolling

Test results: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7960//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7960//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-protocol.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7960//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-thrift.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7960//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-client.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7960//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-examples.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7960//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop1-compat.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7960//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-prefix-tree.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7960//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-common.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7960//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-server.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7960//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop-compat.html
Console output: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7960//console

This message is automatically generated.

 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, HBASE-9261_v4.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)


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

2013-10-07 Thread Anoop Sam John (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-9261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=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)


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

2013-09-20 Thread Anoop Sam John (JIRA)

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

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

[~rajesh23] Do u have an updated patch? Thanks

 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


 These hooks helps for checking Resources(blocking memstore size) and 
 necessary locking on index region while performing batch of mutations. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


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

2013-09-20 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-9261:
--

{color:green}+1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12604224/HBASE-9261_v3.patch
  against trunk revision .

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:green}+1 tests included{color}.  The patch appears to include 6 new 
or modified tests.

{color:green}+1 hadoop1.0{color}.  The patch compiles against the hadoop 
1.0 profile.

{color:green}+1 hadoop2.0{color}.  The patch compiles against the hadoop 
2.0 profile.

{color:green}+1 javadoc{color}.  The javadoc tool did not generate any 
warning messages.

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:green}+1 findbugs{color}.  The patch does not introduce any new 
Findbugs (version 1.3.9) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:green}+1 lineLengths{color}.  The patch does not introduce lines 
longer than 100

  {color:green}+1 site{color}.  The mvn site goal succeeds with this patch.

{color:green}+1 core tests{color}.  The patch passed unit tests in .

Test results: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7319//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7319//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-prefix-tree.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7319//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-client.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7319//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-common.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7319//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-protocol.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7319//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-server.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7319//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop1-compat.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7319//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-examples.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7319//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop-compat.html
Console output: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7319//console

This message is automatically generated.

 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 is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


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

2013-08-31 Thread Anoop Sam John (JIRA)

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

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

Some extra white spaces but its ok, can correct on commit.
Pass the Operation type in postCloseRegionOperation also?
Can the post hooks called for all cases of start/closeRegionOp()?  Generically 
call in startRegionOperation  closeRegionOperation?


 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


 These hooks helps for checking Resources(blocking memstore size) and 
 necessary locking on index region while performing batch of mutations. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


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

2013-08-30 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-9261:
--

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12600771/HBASE-9261_v2.patch
  against trunk revision .

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:green}+1 tests included{color}.  The patch appears to include 6 new 
or modified tests.

{color:green}+1 hadoop1.0{color}.  The patch compiles against the hadoop 
1.0 profile.

{color:green}+1 hadoop2.0{color}.  The patch compiles against the hadoop 
2.0 profile.

{color:green}+1 javadoc{color}.  The javadoc tool did not generate any 
warning messages.

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:green}+1 findbugs{color}.  The patch does not introduce any new 
Findbugs (version 1.3.9) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:green}+1 lineLengths{color}.  The patch does not introduce lines 
longer than 100

  {color:green}+1 site{color}.  The mvn site goal succeeds with this patch.

 {color:red}-1 core tests{color}.  The patch failed these unit tests:
   org.apache.hadoop.hbase.regionserver.TestAtomicOperation

Test results: 
https://builds.apache.org/job/PreCommit-HBASE-Build/6985//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/6985//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-prefix-tree.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/6985//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-client.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/6985//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-common.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/6985//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-protocol.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/6985//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-server.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/6985//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop1-compat.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/6985//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-examples.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/6985//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop-compat.html
Console output: 
https://builds.apache.org/job/PreCommit-HBASE-Build/6985//console

This message is automatically generated.

 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


 These hooks helps for checking Resources(blocking memstore size) and 
 necessary locking on index region while performing batch of mutations. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


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

2013-08-30 Thread Ted Yu (JIRA)

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

Ted Yu commented on HBASE-9261:
---

lgtm

 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


 These hooks helps for checking Resources(blocking memstore size) and 
 necessary locking on index region while performing batch of mutations. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


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

2013-08-25 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-9261:
--

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12599135/HBASE-9261.patch
  against trunk revision .

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:green}+1 tests included{color}.  The patch appears to include 6 new 
or modified tests.

{color:green}+1 hadoop1.0{color}.  The patch compiles against the hadoop 
1.0 profile.

{color:green}+1 hadoop2.0{color}.  The patch compiles against the hadoop 
2.0 profile.

{color:red}-1 javadoc{color}.  The javadoc tool appears to have generated 3 
warning messages.

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:red}-1 findbugs{color}.  The patch appears to introduce 1 new 
Findbugs (version 1.3.9) warnings.

{color:red}-1 release audit{color}.  The applied patch generated 2 release 
audit warnings (more than the trunk's current 0 warnings).

{color:green}+1 lineLengths{color}.  The patch does not introduce lines 
longer than 100

  {color:green}+1 site{color}.  The mvn site goal succeeds with this patch.

{color:green}+1 core tests{color}.  The patch passed unit tests in .

Test results: 
https://builds.apache.org/job/PreCommit-HBASE-Build/6881//testReport/
Release audit warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/6881//artifact/trunk/patchprocess/patchReleaseAuditProblems.txt
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/6881//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-prefix-tree.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/6881//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-client.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/6881//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-common.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/6881//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-protocol.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/6881//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-server.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/6881//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop1-compat.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/6881//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-examples.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/6881//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop-compat.html
Console output: 
https://builds.apache.org/job/PreCommit-HBASE-Build/6881//console

This message is automatically generated.

 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


 These hooks helps for checking Resources(blocking memstore size) and 
 necessary locking on index region while performing batch of mutations. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


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

2013-08-23 Thread Andrew Purtell (JIRA)

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

Andrew Purtell commented on HBASE-9261:
---

Add javadoc that postStartRegionOperation and postCloseRegionOperation don't 
allow bypass and lgtm.

 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


 These hooks helps for checking Resources(blocking memstore size) and 
 necessary locking on index region while performing batch of mutations. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


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

2013-08-23 Thread Ted Yu (JIRA)

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

Ted Yu commented on HBASE-9261:
---

When isReplay is true, there would be a postCloseRegionOperation() call which 
is not paired with corresponding postStartRegionOperation() call.

Should postStartRegionOperation() be passed Operation enum ? That way the two 
hooks would be paired.

 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


 These hooks helps for checking Resources(blocking memstore size) and 
 necessary locking on index region while performing batch of mutations. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


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

2013-08-21 Thread stack (JIRA)

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

stack commented on HBASE-9261:
--

Patch lgtm (I like how these changes are coming in in digestible pieces).  
Anyone else want to looksee?

 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
 Attachments: HBASE-9261.patch


 These hooks helps for checking Resources(blocking memstore size) and 
 necessary locking on index region while performing batch of mutations. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira