NihalJain commented on code in PR #6781:
URL: https://github.com/apache/hbase/pull/6781#discussion_r2028544879
##########
hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java:
##########
@@ -1513,7 +1518,16 @@ private boolean waitForNamespaceOnline() throws
IOException {
}
return true;
}
+ private void appendZkAclToMasterCoprocessorConf(Configuration conf) {
+ String plugins = conf.get(CoprocessorHost.MASTER_COPROCESSOR_CONF_KEY, "");
+ String zkAclClassName = ZKAclUpdaterCoprocessor.class.getCanonicalName();
+ String accessControllerClassName = conf.get(SECURITY_COPROCESSOR_CONF_KEY,
DEFAULT_SECURITY_COPROCESSOR_CONF_NAME);
+ if ((plugins.contains(accessControllerClassName) ) &&
!plugins.contains(zkAclClassName)) {
+ conf.set(CoprocessorHost.MASTER_COPROCESSOR_CONF_KEY,
+ (plugins.equals("") ? "" : (plugins + ",")) +
ZKAclUpdaterCoprocessor.class.getCanonicalName() );
Review Comment:
ZKAclUpdaterCoprocessor can be placed first so that we are sure zk acl is
initialed before AccessController loads?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
