Skip to content

Commit 11f3e1f

Browse files
Mixficsolwuxianrong
andauthored
added the correct loading of admin-cmd-list in the configuration file (OpenAtomFoundation#3077)
Co-authored-by: wuxianrong <[email protected]>
1 parent f447809 commit 11f3e1f

2 files changed

Lines changed: 2 additions & 5 deletions

File tree

conf/pika.conf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ slow-cmd-list :
5151
# List of commands considered as administrative. These commands will be handled by the admin thread pool. Modify this list as needed.
5252
# Default commands: info, ping, monitor
5353
# This parameter is only supported by the CONFIG GET command and not by CONFIG SET.
54-
admin-cmd-list : info, ping, monitor
54+
admin-cmd-list : info, ping, monitor, auth, config
5555

5656
# The number of threads to write DB in slaveNode when replicating.
5757
# It's preferable to set slave's sync-thread-num value close to master's thread-pool-size.

src/pika_conf.cc

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -192,10 +192,7 @@ int PikaConf::Load() {
192192

193193
std::string admin_cmd_list;
194194
GetConfStr("admin-cmd-list", &admin_cmd_list);
195-
if (admin_cmd_list == "") {
196-
admin_cmd_list = "info, monitor, ping";
197-
SetAdminCmd(admin_cmd_list);
198-
}
195+
SetAdminCmd(admin_cmd_list);
199196

200197
std::string unfinished_full_sync;
201198
GetConfStr("internal-used-unfinished-full-sync", &unfinished_full_sync);

0 commit comments

Comments
 (0)