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

2013-12-15 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-9261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13848734#comment-13848734 ] Hudson commented on HBASE-9261: --- FAILURE: Integrated in HBase-TRUNK-on-Hadoop-1.1 #7 (See

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

2013-12-14 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-9261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13848528#comment-13848528 ] Hudson commented on HBASE-9261: --- SUCCESS: Integrated in HBase-TRUNK #4726 (See

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

2013-12-14 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-9261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13848532#comment-13848532 ] Hudson commented on HBASE-9261: --- SUCCESS: Integrated in HBase-0.98 #13 (See

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

2013-12-14 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-9261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13848544#comment-13848544 ] Hudson commented on HBASE-9261: --- FAILURE: Integrated in HBase-0.98-on-Hadoop-1.1 #10 (See

[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 ?

[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

[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

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

2013-12-13 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-9261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13847480#comment-13847480 ] Hadoop QA commented on HBASE-9261: -- {color:red}-1 overall{color}. Here are the results

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

2013-12-13 Thread Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-9261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13847676#comment-13847676 ] Ted Yu commented on HBASE-9261: --- From test output for

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

2013-12-13 Thread Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-9261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13847712#comment-13847712 ] Ted Yu commented on HBASE-9261: --- Good catch, Anoop - I was wonder why my patch was 1KB

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

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=13847710#comment-13847710 ] Anoop Sam John commented on HBASE-9261: --- Changes in OpenRegionHandler is not needed

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

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=13847723#comment-13847723 ] Anoop Sam John commented on HBASE-9261: --- Ted's patch LGTM.. Add cp hooks after

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

2013-12-13 Thread rajeshbabu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-9261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13847740#comment-13847740 ] rajeshbabu commented on HBASE-9261: --- Thank you very much Ted for correcting it. Add cp

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

2013-12-13 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-9261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13847853#comment-13847853 ] Hadoop QA commented on HBASE-9261: -- {color:red}-1 overall{color}. Here are the results

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

2013-12-13 Thread rajeshbabu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-9261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13848163#comment-13848163 ] rajeshbabu commented on HBASE-9261: --- I will commit this today if no object. Add cp

[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

[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

[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.

[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

[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

[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

[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

[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

[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

[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?

[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

[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

[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

[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

[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

[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

[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

[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