[HBASE-12277] Refactor bulkLoad methods in AccessController to its own interface

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.98.9, 0.99.2
• Hadoop Flags: Reviewed
• Release Note: Adds new BulkLoadObserver Interface
• Assignee: Madhan Neethiraj
• Reporter: Madhan Neethiraj
• Created: 15/Oct/14 23:00
• Updated: 06/Apr/18 17:55
• Resolved: 19/Nov/14 03:07

Description

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.