INNOCENT-BOY commented on a change in pull request #8314: URL: https://github.com/apache/pinot/pull/8314#discussion_r828006135
########## File path: pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/PinotHelixResourceManager.java ########## @@ -1268,6 +1274,32 @@ public Schema getSchemaForTableConfig(TableConfig tableConfig) { AccessOption.PERSISTENT); } + public void initUserACLConfig() throws IOException { + if (Optional.ofNullable(ZKMetadataProvider.getAllUserName(_propertyStore)).isEmpty()) { + addUser(new UserConfig("admin", "Pda@wap%admin", ComponentType.CONTROLLER.name(), + RoleType.ADMIN.name(), null, null)); + addUser(new UserConfig("admin", "Pda@wap%admin", ComponentType.BROKER.name(), + RoleType.ADMIN.name(), null, null)); + addUser(new UserConfig("admin", "Pda@wap%admin", ComponentType.SERVER.name(), + RoleType.ADMIN.name(), null, null)); + } Review comment: @apucher You are right. I have set DEFAULT_ACCESS_CONTROL_USERNAME and DEFAULT_ACCESS_CONTROL_PASSWORD. These two variables have default value. And user can set these two value via controller configuration file. -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org