You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Francis Liu (JIRA)" <ji...@apache.org> on 2012/06/18 03:42:42 UTC

[jira] [Created] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Francis Liu created HBASE-6224:
----------------------------------

             Summary: add Pre and Post coprocessor hooks for BulkLoad
                 Key: HBASE-6224
                 URL: https://issues.apache.org/jira/browse/HBASE-6224
             Project: HBase
          Issue Type: Sub-task
            Reporter: Francis Liu
            Assignee: Francis Liu




--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Andrew Purtell (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Andrew Purtell updated HBASE-6224:
----------------------------------

    Attachment: HBASE-6224-apurtell-0.94.patch
    
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224-apurtell-0.94.patch, HBASE-6224-apurtell.patch, HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Hadoop QA (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13393649#comment-13393649 ] 

Hadoop QA commented on HBASE-6224:
----------------------------------

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12532357/HBASE-6224.patch
  against trunk revision .

    +1 @author.  The patch does not contain any @author tags.

    +1 tests included.  The patch appears to include 6 new or modified tests.

    +1 hadoop2.0.  The patch compiles against the hadoop 2.0 profile.

    +1 javadoc.  The javadoc tool did not generate any warning messages.

    +1 javac.  The applied patch does not increase the total number of javac compiler warnings.

    -1 findbugs.  The patch appears to introduce 7 new Findbugs (version 1.3.9) warnings.

    +1 release audit.  The applied patch does not increase the total number of release audit warnings.

    +1 core tests.  The patch passed unit tests in .

Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/2179//testReport/
Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/2179//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-server.html
Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/2179//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-common.html
Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/2179//console

This message is automatically generated.
                
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Andrew Purtell (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Andrew Purtell updated HBASE-6224:
----------------------------------

    Comment: was deleted

(was: -1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12532357/HBASE-6224.patch
  against trunk revision .

    +1 @author.  The patch does not contain any @author tags.

    +1 tests included.  The patch appears to include 6 new or modified tests.

    +1 hadoop2.0.  The patch compiles against the hadoop 2.0 profile.

    +1 javadoc.  The javadoc tool did not generate any warning messages.

    +1 javac.  The applied patch does not increase the total number of javac compiler warnings.

    -1 findbugs.  The patch appears to introduce 7 new Findbugs (version 1.3.9) warnings.

    +1 release audit.  The applied patch does not increase the total number of release audit warnings.

    +1 core tests.  The patch passed unit tests in .

Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/2179//testReport/
Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/2179//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-server.html
Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/2179//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-common.html
Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/2179//console

This message is automatically generated.)
    
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Andrew Purtell (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13399650#comment-13399650 ] 

Andrew Purtell commented on HBASE-6224:
---------------------------------------

@stack Ok. 
                
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224-apurtell.patch, HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Francis Liu (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13399770#comment-13399770 ] 

Francis Liu commented on HBASE-6224:
------------------------------------

Is "AccessController v2" already in a usable state in 0.94? If not what are the patches that need to go in?
                
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224-apurtell-0.94.patch, HBASE-6224-apurtell.patch, HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Andrew Purtell (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Andrew Purtell updated HBASE-6224:
----------------------------------

    Attachment:     (was: HBASE-6224-apurtell.patch)
    
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224-apurtell.patch, HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Andrew Purtell (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13397918#comment-13397918 ] 

Andrew Purtell commented on HBASE-6224:
---------------------------------------

This is the patch on https://reviews.apache.org/r/5380/ right? 

+1 if so.
                
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Andrew Purtell (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Andrew Purtell updated HBASE-6224:
----------------------------------

    Attachment: HBASE-6224-apurtell.patch
    
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224-apurtell.patch, HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Francis Liu (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Francis Liu updated HBASE-6224:
-------------------------------

    Attachment: HBASE-6224.patch

After a discussion with Andy during the HBase BOF. I simplified the signature to just pass the CF-Path pair and let the user create the StoreFile instance if he needs to. I've also added a post hook.
                
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13399918#comment-13399918 ] 

Hudson commented on HBASE-6224:
-------------------------------

Integrated in HBase-TRUNK-on-Hadoop-2.0.0 #66 (See [https://builds.apache.org/job/HBase-TRUNK-on-Hadoop-2.0.0/66/])
    HBASE-6224. Add pre and post coprocessor hooks for bulk load (Francis Liu) (Revision 1353057)

     Result = FAILURE
apurtell : 
Files : 
* /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/coprocessor/BaseRegionObserver.java
* /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/coprocessor/RegionObserver.java
* /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
* /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RegionCoprocessorHost.java
* /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java
* /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFileScanner.java
* /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/coprocessor/SimpleRegionObserver.java
* /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/coprocessor/TestRegionObserverInterface.java

                
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>             Fix For: 0.96.0, 0.94.1
>
>         Attachments: HBASE-6224-apurtell-0.94.patch, HBASE-6224-apurtell.patch, HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Francis Liu (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13399616#comment-13399616 ] 

Francis Liu commented on HBASE-6224:
------------------------------------

@Andy, looks good to me. Good catch on the bypass and loaded. Should I come up with another patch?
                
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224-apurtell.patch, HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Andrew Purtell (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Andrew Purtell updated HBASE-6224:
----------------------------------

    Comment: was deleted

(was: Integrated in HBase-TRUNK #3061 (See [https://builds.apache.org/job/HBase-TRUNK/3061/])
    HBASE-6224. Add pre and post coprocessor hooks for bulk load (Francis Liu) (Revision 1353057)

     Result = FAILURE
apurtell : 
Files : 
* /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/coprocessor/BaseRegionObserver.java
* /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/coprocessor/RegionObserver.java
* /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
* /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RegionCoprocessorHost.java
* /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java
* /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFileScanner.java
* /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/coprocessor/SimpleRegionObserver.java
* /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/coprocessor/TestRegionObserverInterface.java
)
    
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>             Fix For: 0.96.0, 0.94.1
>
>         Attachments: HBASE-6224-apurtell-0.94.patch, HBASE-6224-apurtell.patch, HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Comment Edited] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Andrew Purtell (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13397918#comment-13397918 ] 

Andrew Purtell edited comment on HBASE-6224 at 6/21/12 12:57 AM:
-----------------------------------------------------------------

This is the patch on https://reviews.apache.org/r/5380/ right? 

-+1 if so.-

Edit: Found issues on re-review.
                
      was (Author: apurtell):
    This is the patch on https://reviews.apache.org/r/5380/ right? 

+1 if so.
                  
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Andrew Purtell (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13399625#comment-13399625 ] 

Andrew Purtell commented on HBASE-6224:
---------------------------------------

@Francis, not unless you want to add more. Otherwise I'll commit the -apurtell patch in a few hours. Thanks for the ltgm.
                
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224-apurtell.patch, HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13399774#comment-13399774 ] 

Hudson commented on HBASE-6224:
-------------------------------

Integrated in HBase-TRUNK #3061 (See [https://builds.apache.org/job/HBase-TRUNK/3061/])
    HBASE-6224. Add pre and post coprocessor hooks for bulk load (Francis Liu) (Revision 1353057)

     Result = FAILURE
apurtell : 
Files : 
* /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/coprocessor/BaseRegionObserver.java
* /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/coprocessor/RegionObserver.java
* /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
* /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RegionCoprocessorHost.java
* /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java
* /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFileScanner.java
* /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/coprocessor/SimpleRegionObserver.java
* /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/coprocessor/TestRegionObserverInterface.java

                
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224-apurtell-0.94.patch, HBASE-6224-apurtell.patch, HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Andrew Purtell (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Andrew Purtell updated HBASE-6224:
----------------------------------

    Comment: was deleted

(was: -1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12533145/HBASE-6224-apurtell-0.94.patch
  against trunk revision .

    +1 @author.  The patch does not contain any @author tags.

    +1 tests included.  The patch appears to include 6 new or modified tests.

    -1 patch.  The patch command could not apply the patch.

Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/2230//console

This message is automatically generated.)
    
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>             Fix For: 0.96.0, 0.94.1
>
>         Attachments: HBASE-6224-apurtell-0.94.patch, HBASE-6224-apurtell.patch, HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Andrew Purtell (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13398044#comment-13398044 ] 

Andrew Purtell commented on HBASE-6224:
---------------------------------------

@Francis, if you are agreeable to my modifications to your patch, we can commit this.
                
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224-apurtell.patch, HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Francis Liu (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13399696#comment-13399696 ] 

Francis Liu commented on HBASE-6224:
------------------------------------

@Andy, nope nothing to add. BTW can we backport this to 0.94 and 0.92? We're still mulling over wether to pickup 0.92 or 0.94. Though it would be good to get bulkload security covered in the current stable release anyway?
                
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224-apurtell.patch, HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Francis Liu (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Francis Liu updated HBASE-6224:
-------------------------------

    Status: Patch Available  (was: Open)
    
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13400013#comment-13400013 ] 

Hudson commented on HBASE-6224:
-------------------------------

Integrated in HBase-0.94-security #37 (See [https://builds.apache.org/job/HBase-0.94-security/37/])
    HBASE-6224. Add pre and post coprocessor hooks for bulk load (Francis Liu) (Revision 1353060)

     Result = SUCCESS
apurtell : 
Files : 
* /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/coprocessor/BaseRegionObserver.java
* /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/coprocessor/RegionObserver.java
* /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
* /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/RegionCoprocessorHost.java
* /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java
* /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFileScanner.java
* /hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/coprocessor/SimpleRegionObserver.java
* /hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/coprocessor/TestRegionObserverInterface.java

                
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>             Fix For: 0.96.0, 0.94.1
>
>         Attachments: HBASE-6224-apurtell-0.94.patch, HBASE-6224-apurtell.patch, HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Hadoop QA (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13399788#comment-13399788 ] 

Hadoop QA commented on HBASE-6224:
----------------------------------

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12533145/HBASE-6224-apurtell-0.94.patch
  against trunk revision .

    +1 @author.  The patch does not contain any @author tags.

    +1 tests included.  The patch appears to include 6 new or modified tests.

    -1 patch.  The patch command could not apply the patch.

Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/2230//console

This message is automatically generated.
                
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224-apurtell-0.94.patch, HBASE-6224-apurtell.patch, HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Closed] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Lars Hofhansl (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Lars Hofhansl closed HBASE-6224.
--------------------------------

    
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: Coprocessors
>    Affects Versions: 0.94.1, 0.96.0
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>             Fix For: 0.94.1, 0.96.0
>
>         Attachments: HBASE-6224-apurtell-0.94.patch, HBASE-6224-apurtell.patch, HBASE-6224.patch
>
>


--
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] [Updated] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Andrew Purtell (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Andrew Purtell updated HBASE-6224:
----------------------------------

    Attachment: HBASE-6224-apurtell.patch

Attached patch that addresses my re-review comments. 

Here's the functional difference:

{code}
Index: hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
===================================================================
--- hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java	(revision 1352336)
+++ hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java	(working copy)
@@ -3122,10 +3122,20 @@
       HRegion region = getRegion(request.getRegion());
       List<Pair<byte[], String>> familyPaths = new ArrayList<Pair<byte[], String>>();
       for (FamilyPath familyPath: request.getFamilyPathList()) {
-        familyPaths.add(new Pair<byte[], String>(
-          familyPath.getFamily().toByteArray(), familyPath.getPath()));
+        familyPaths.add(new Pair<byte[], String>(familyPath.getFamily().toByteArray(),
+          familyPath.getPath()));
       }
-      boolean loaded = region.bulkLoadHFiles(familyPaths);
+      boolean bypass = false;
+      if (region.getCoprocessorHost() != null) {
+        bypass = region.getCoprocessorHost().preBulkLoadHFile(familyPaths);
+      }
+      boolean loaded = false;
+      if (!bypass) {
+        loaded = region.bulkLoadHFiles(familyPaths);
+      }
+      if (region.getCoprocessorHost() != null) {
+        loaded = region.getCoprocessorHost().postBulkLoadHFile(familyPaths, loaded);
+      }
       BulkLoadHFileResponse.Builder builder = BulkLoadHFileResponse.newBuilder();
       builder.setLoaded(loaded);
       return builder.build();
{code}
                
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224-apurtell.patch, HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Andrew Purtell (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Andrew Purtell updated HBASE-6224:
----------------------------------

    Comment: was deleted

(was: -1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12532357/HBASE-6224.patch
  against trunk revision .

    +1 @author.  The patch does not contain any @author tags.

    +1 tests included.  The patch appears to include 6 new or modified tests.

    +1 hadoop2.0.  The patch compiles against the hadoop 2.0 profile.

    +1 javadoc.  The javadoc tool did not generate any warning messages.

    +1 javac.  The applied patch does not increase the total number of javac compiler warnings.

    -1 findbugs.  The patch appears to introduce 7 new Findbugs (version 1.3.9) warnings.

    +1 release audit.  The applied patch does not increase the total number of release audit warnings.

    +1 core tests.  The patch passed unit tests in .

Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/2180//testReport/
Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/2180//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-common.html
Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/2180//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-server.html
Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/2180//console

This message is automatically generated.)
    
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Andrew Purtell (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13399715#comment-13399715 ] 

Andrew Purtell commented on HBASE-6224:
---------------------------------------

@Francis These hooks can go back to 0.92 IMHO, no problem there. However, we've not been committing "AccessController v2" changes to 0.92, only 0.94 and trunk. Seems prudent to leave 0.92 as "AccessController v1". Therefore I'd recommend you pick up 0.94.
                
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224-apurtell.patch, HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Andrew Purtell (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Andrew Purtell updated HBASE-6224:
----------------------------------

          Component/s: coprocessors
    Affects Version/s: 0.94.1
                       0.96.0
    
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Hadoop QA (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13393652#comment-13393652 ] 

Hadoop QA commented on HBASE-6224:
----------------------------------

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12532357/HBASE-6224.patch
  against trunk revision .

    +1 @author.  The patch does not contain any @author tags.

    +1 tests included.  The patch appears to include 6 new or modified tests.

    +1 hadoop2.0.  The patch compiles against the hadoop 2.0 profile.

    +1 javadoc.  The javadoc tool did not generate any warning messages.

    +1 javac.  The applied patch does not increase the total number of javac compiler warnings.

    -1 findbugs.  The patch appears to introduce 7 new Findbugs (version 1.3.9) warnings.

    +1 release audit.  The applied patch does not increase the total number of release audit warnings.

    +1 core tests.  The patch passed unit tests in .

Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/2180//testReport/
Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/2180//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-common.html
Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/2180//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-server.html
Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/2180//console

This message is automatically generated.
                
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Andrew Purtell (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Andrew Purtell updated HBASE-6224:
----------------------------------

       Resolution: Fixed
    Fix Version/s: 0.94.1
                   0.96.0
     Hadoop Flags: Reviewed
           Status: Resolved  (was: Patch Available)

Committed to trunk and 0.94 branches. Tests pass locally. Thanks for the patch Francis!
                
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>             Fix For: 0.96.0, 0.94.1
>
>         Attachments: HBASE-6224-apurtell-0.94.patch, HBASE-6224-apurtell.patch, HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "stack (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13399641#comment-13399641 ] 

stack commented on HBASE-6224:
------------------------------

Patch looks good.  Its a pity StoreFile has to be made public but at list its limited private.  Fix this on commit:

{code}
+   * @param familyPaths family and path pairs to be bulk loaded
+   * @param hasLoaded whether load was successful or not
+   * @return
+   * @throws IOException
+   */
{code}

Add something for return since this is a class that will be implemented (Maybe say what the pair Paths are too?  StoreFiles?)

Else lgtm.  +1


                
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224-apurtell.patch, HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13399817#comment-13399817 ] 

Hudson commented on HBASE-6224:
-------------------------------

Integrated in HBase-0.94 #274 (See [https://builds.apache.org/job/HBase-0.94/274/])
    HBASE-6224. Add pre and post coprocessor hooks for bulk load (Francis Liu) (Revision 1353060)

     Result = FAILURE
apurtell : 
Files : 
* /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/coprocessor/BaseRegionObserver.java
* /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/coprocessor/RegionObserver.java
* /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
* /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/RegionCoprocessorHost.java
* /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java
* /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFileScanner.java
* /hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/coprocessor/SimpleRegionObserver.java
* /hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/coprocessor/TestRegionObserverInterface.java

                
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>             Fix For: 0.96.0, 0.94.1
>
>         Attachments: HBASE-6224-apurtell-0.94.patch, HBASE-6224-apurtell.patch, HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-6224) add Pre and Post coprocessor hooks for BulkLoad

Posted by "Andrew Purtell (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13399772#comment-13399772 ] 

Andrew Purtell commented on HBASE-6224:
---------------------------------------

bq. Is "AccessController v2" already in a usable state in 0.94? 

Yes, but it's under development (in a backwards compatible way). See umbrella issue HBASE-6096.
                
> add Pre and Post coprocessor hooks for BulkLoad
> -----------------------------------------------
>
>                 Key: HBASE-6224
>                 URL: https://issues.apache.org/jira/browse/HBASE-6224
>             Project: HBase
>          Issue Type: Sub-task
>          Components: coprocessors
>    Affects Versions: 0.96.0, 0.94.1
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>         Attachments: HBASE-6224-apurtell-0.94.patch, HBASE-6224-apurtell.patch, HBASE-6224.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira