diff --git a/be/src/common/config.cpp b/be/src/common/config.cpp index a773e1a8bfedf5..8055e3fe456d6b 100644 --- a/be/src/common/config.cpp +++ b/be/src/common/config.cpp @@ -1732,11 +1732,18 @@ std::vector> get_config_info() { std::vector _config; _config.push_back(it.first); + std::string config_val = it.second; + // For compatibility, this PR #32933 change the log dir's config logic, + // and deprecate the `sys_log_dir` config. + if (it.first == "sys_log_dir" && config_val == "") { + config_val = fmt::format("{}/log", std::getenv("DORIS_HOME")); + } + _config.emplace_back(field_it->second.type); if (0 == strcmp(field_it->second.type, "bool")) { - _config.emplace_back(it.second == "1" ? "true" : "false"); + _config.emplace_back(config_val == "1" ? "true" : "false"); } else { - _config.push_back(it.second); + _config.push_back(config_val); } _config.emplace_back(field_it->second.valmutable ? "true" : "false"); 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 dd66fdcb0f593b..3306f314f7da75 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 @@ -376,7 +376,13 @@ public static synchronized List> getConfigInfo(PatternMatcher match if (matcher == null || matcher.match(confKey)) { List config = Lists.newArrayList(); config.add(confKey); - config.add(getConfValue(f)); + String value = getConfValue(f); + // For compatibility, this PR #32933 change the log dir's config logic, + // and deprecate the `sys_log_dir` config. + if (confKey.equals("sys_log_dir") && Strings.isNullOrEmpty(value)) { + value = System.getenv("DORIS_HOME") + "/log"; + } + config.add(value); config.add(f.getType().getSimpleName()); config.add(String.valueOf(confField.mutable())); config.add(String.valueOf(confField.masterOnly()));