[ https://issues.apache.org/jira/browse/HBASE-12277?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14183066#comment-14183066 ]
Hudson commented on HBASE-12277: -------------------------------- SUCCESS: Integrated in HBase-TRUNK #5696 (See [https://builds.apache.org/job/HBase-TRUNK/5696/]) HBASE-12277 Refactor bulkLoad methods in AccessController to its own interface (Madhan Neethiraj) (stack: rev 2916d4f3568184f92006dba9a1e4ef18492643ea) * hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/SecureBulkLoadEndpoint.java * hbase-server/src/main/java/org/apache/hadoop/hbase/coprocessor/BulkLoadObserver.java * hbase-server/src/test/java/org/apache/hadoop/hbase/coprocessor/TestClassLoading.java * hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/AccessController.java * hbase-server/src/main/java/org/apache/hadoop/hbase/coprocessor/CoprocessorHost.java > Refactor bulkLoad methods in AccessController to its own interface > ------------------------------------------------------------------ > > Key: HBASE-12277 > URL: https://issues.apache.org/jira/browse/HBASE-12277 > Project: HBase > Issue Type: Bug > Reporter: Madhan Neethiraj > Fix For: 2.0.0, 0.99.2 > > Attachments: > 0001-HBASE-12277-Refactored-bulk-load-methods-from-Access.patch, > 0002-HBASE-12277-License-text-added-to-the-newly-created-.patch, > HBASE-12277-v2.patch, HBASE-12277-v3.patch, HBASE-12277-v4.patch, > HBASE-12277.patch > > > SecureBulkLoadEndPoint references couple of methods, prePrepareBulkLoad() and > preCleanupBulkLoad(), implemented in AccessController i.e. direct coupling > between AccessController and SecureBuikLoadEndPoint classes. > SecureBulkLoadEndPoint assumes presence of AccessController in > secure-cluster. If HBase is configured with another coprocessor for > access-control, SecureBulkLoadEndPoint fails with NPE. > To remove this direct coupling, bulk-load related methods in AccessController > should be refactored to an interface; and have AccessController implement > this interfaces. SecureBulkLoadEndPoint should then look for coprocessors > that implement this interface, instead of directly looking for > AccessController. -- This message was sent by Atlassian JIRA (v6.3.4#6332)