[HBASE-9897] Clean up some security configuration checks in LoadIncrementalHFiles

• Type: Task
• Status: Closed
• Priority: Major
• Resolution: Incomplete
• Component/s: security
• Assignee: Unassigned
• Reporter: Gary Helmling
• Created: 05/Nov/13 22:56
• Updated: 16/Jun/22 18:28
• Resolved: 16/Jun/22 18:28

Description

In LoadIncrementalHFiles, use of SecureBulkLoadClient is conditioned on UserProvider.isHBaseSecurityEnabled() in a couple of places. However, use of secure bulk loading seems to be required more by use of HDFS secure authentication, instead of HBase secure authentication. It should be possible to use secure bulk loading, as long as SecureBulkLoadEndpoint is loaded, and HDFS secure authentication is enabled, regardless of the HBase authentication configuration.

In addition, SecureBulkLoadEndpoint does a direct check on permissions by referencing AccessController loaded on the same region, i.e.:

      getAccessController().prePrepareBulkLoad(env);

It seems like this will throw an NPE if AccessController is not configured. We need an additional null check to handle this case gracefully.

Issue Links

• relates to: HBASE-9890 MR jobs are not working if started by a delegated user