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

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


The following commit(s) were added to refs/heads/master by this push:
     new 776563dd1c3 [improve](config) Report config name and value if value is 
invalid (#34945)
776563dd1c3 is described below

commit 776563dd1c35761a90e44f4df3d9762d94ada493
Author: walter <w41te...@gmail.com>
AuthorDate: Sat May 18 14:24:56 2024 +0800

    [improve](config) Report config name and value if value is invalid (#34945)
---
 fe/fe-common/src/main/java/org/apache/doris/common/ConfigBase.java | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/fe/fe-common/src/main/java/org/apache/doris/common/ConfigBase.java 
b/fe/fe-common/src/main/java/org/apache/doris/common/ConfigBase.java
index 3306f314f7d..21bd84ebd72 100644
--- a/fe/fe-common/src/main/java/org/apache/doris/common/ConfigBase.java
+++ b/fe/fe-common/src/main/java/org/apache/doris/common/ConfigBase.java
@@ -224,7 +224,12 @@ public class ConfigBase {
                 continue;
             }
 
-            setConfigField(f, confVal);
+            try {
+                setConfigField(f, confVal);
+            } catch (Exception e) {
+                String msg = String.format("Failed to set config, name: %s, 
value: %s", f.getName(), confVal);
+                throw new IllegalArgumentException(msg, e);
+            }
         }
     }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to