From 345cd965ef3965ec2dfd5dd5c9f8aa65fa5bba4d Mon Sep 17 00:00:00 2001
From: cambyzhu <cambyzhu@tencent.com>
Date: Wed, 6 Aug 2025 19:26:07 +0800
Subject: [PATCH] opt ADMIN SET ALL FRONTENDS CONFIG

---
 .../commands/AdminSetFrontendConfigCommand.java      | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/AdminSetFrontendConfigCommand.java b/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/AdminSetFrontendConfigCommand.java
index 3c7e8d282f2d34..a3a5e3346bc378 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/AdminSetFrontendConfigCommand.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/AdminSetFrontendConfigCommand.java
@@ -59,11 +59,13 @@ public AdminSetFrontendConfigCommand(NodeType type, Map<String, String> configs,
         }
         this.applyToAll = applyToAll;
 
-        // we have to analyze configs here to determine whether to forward it to master
-        for (String key : this.configs.keySet()) {
-            if (ConfigBase.checkIsMasterOnly(key)) {
-                redirectStatus = RedirectStatus.FORWARD_NO_SYNC;
-                this.applyToAll = false;
+        if (!this.applyToAll) {
+            // we have to analyze configs here to determine whether to forward it to master
+            for (String key : this.configs.keySet()) {
+                if (ConfigBase.checkIsMasterOnly(key)) {
+                    redirectStatus = RedirectStatus.FORWARD_NO_SYNC;
+                    break;
+                }
             }
         }
     }