This is an automated email from the ASF dual-hosted git repository.

dataroaring pushed a commit to branch branch-3.0
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/branch-3.0 by this push:
     new b26cba1247c branch-3.0: [fix](config) dynamic set FE masterOnly config 
will lost after change MASTER #54409 (#54601)
b26cba1247c is described below

commit b26cba1247c32ed97a7ad73a0afad97094880fb5
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Fri Aug 15 21:21:05 2025 +0800

    branch-3.0: [fix](config) dynamic set FE masterOnly config will lost after 
change MASTER #54409 (#54601)
    
    Cherry-picked from #54409
    
    Co-authored-by: camby <[email protected]>
---
 .../java/org/apache/doris/analysis/AdminSetConfigStmt.java   | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/analysis/AdminSetConfigStmt.java 
b/fe/fe-core/src/main/java/org/apache/doris/analysis/AdminSetConfigStmt.java
index 458116924d7..b197c6c2bca 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/analysis/AdminSetConfigStmt.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/analysis/AdminSetConfigStmt.java
@@ -49,11 +49,13 @@ public class AdminSetConfigStmt extends DdlStmt implements 
NotFallbackInParser {
         }
         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;
+                }
             }
         }
     }


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to