This is an automated email from the ASF dual-hosted git repository.
yiguolei pushed a commit to branch branch-2.1
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-2.1 by this push:
new 65f98b13558 branch-2.1: [fix](config) dynamic set FE masterOnly config
will lost after change MASTER #54409 (#54602)
65f98b13558 is described below
commit 65f98b13558dbfa5ffac030c6d97697332548c46
Author: github-actions[bot]
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Wed Aug 13 10:06:27 2025 +0800
branch-2.1: [fix](config) dynamic set FE masterOnly config will lost after
change MASTER #54409 (#54602)
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 1d2e22ee878..8960fa625e3 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
@@ -53,11 +53,13 @@ public class AdminSetConfigStmt extends DdlStmt {
}
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]
