[fix](log4j) fix some issues when modify log config (#21099)

Co-authored-by: caiconghui1 <caiconghui1@jd.com>
apache · Jul 3, 2023 · 4296058 · 4296058
1 parent 76a8c9c
commit 4296058
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 7 deletions.
diff --git a/fe/fe-core/src/main/java/org/apache/doris/common/Log4jConfig.java b/fe/fe-core/src/main/java/org/apache/doris/common/Log4jConfig.java
@@ -47,7 +47,7 @@ public class Log4jConfig extends XmlConfiguration {
             + "        <Pattern>%d{yyyy-MM-dd HH:mm:ss,SSS} %p (%t|%tid)<!--REPLACED BY LOG FORMAT-->%m%n</Pattern>\n"
             + "      </PatternLayout>\n"
             + "    </Console>"
-            + "    <RollingFile name=\"Sys\" fileName=\"${sys_log_dir}/fe.log\" filePattern=\"${sys_log_dir}/fe.log.${sys_file_pattern}-%i\">\n"
+            + "    <RollingFile name=\"Sys\" fileName=\"${sys_log_dir}/fe.log\" filePattern=\"${sys_log_dir}/fe.log.${sys_file_pattern}-%i\" immediateFlush=\"${immediate_flush_flag}\">\n"
             + "      <PatternLayout charset=\"UTF-8\">\n"
             + "        <Pattern>%d{yyyy-MM-dd HH:mm:ss,SSS} %p (%t|%tid)<!--REPLACED BY LOG FORMAT-->%m%n</Pattern>\n"
             + "      </PatternLayout>\n"
@@ -62,7 +62,7 @@ public class Log4jConfig extends XmlConfiguration {
             + "        </Delete>\n"
             + "      </DefaultRolloverStrategy>\n"
             + "    </RollingFile>\n"
-            + "    <RollingFile name=\"SysWF\" fileName=\"${sys_log_dir}/fe.warn.log\" filePattern=\"${sys_log_dir}/fe.warn.log.${sys_file_pattern}-%i\">\n"
+            + "    <RollingFile name=\"SysWF\" fileName=\"${sys_log_dir}/fe.warn.log\" filePattern=\"${sys_log_dir}/fe.warn.log.${sys_file_pattern}-%i\" immediateFlush=\"${immediate_flush_flag}\">\n"
             + "      <PatternLayout charset=\"UTF-8\">\n"
             + "        <Pattern>%d{yyyy-MM-dd HH:mm:ss,SSS} %p (%t|%tid)<!--REPLACED BY LOG FORMAT-->%m%n</Pattern>\n"
             + "      </PatternLayout>\n"
@@ -94,7 +94,7 @@ public class Log4jConfig extends XmlConfiguration {
             + "    </RollingFile>\n"
             + "  </Appenders>\n"
             + "  <Loggers>\n"
-            + "    <Root level=\"${sys_log_level}\" includeLocation=\"${include_location_flag}\" immediateFlush=\"${immediate_flush_flag}\">\n"
+            + "    <Root level=\"${sys_log_level}\" includeLocation=\"${include_location_flag}\">\n"
             + "      <AppenderRef ref=\"Sys\"/>\n"
             + "      <AppenderRef ref=\"SysWF\" level=\"WARN\"/>\n"
             + "      <!--REPLACED BY Console Logger-->\n"

diff --git a/fe/fe-core/src/main/java/org/apache/doris/httpv2/controller/LogController.java b/fe/fe-core/src/main/java/org/apache/doris/httpv2/controller/LogController.java
@@ -90,7 +90,7 @@ private void appendLogConf(Map<String, Map<String, String>> content) {
             }
             if (!Strings.isNullOrEmpty(delVerboseName)) {
                 delVerboseName = delVerboseName.trim();
-                List<String> verboseNames = Lists.newArrayList(configs.y);
+                List<String> verboseNames = Lists.newArrayList(configs.z);
                 if (verboseNames.contains(delVerboseName)) {
                     verboseNames.remove(delVerboseName);
                     configs = Log4jConfig.updateLogging(null, null,
@@ -100,8 +100,9 @@ private void appendLogConf(Map<String, Map<String, String>> content) {
             }
 
             map.put("Level", configs.x);
-            map.put("VerboseNames", StringUtils.join(configs.y, ","));
-            map.put("AuditNames", StringUtils.join(configs.z, ","));
+            map.put("Mode", configs.y);
+            map.put("VerboseNames", StringUtils.join(configs.z, ","));
+            map.put("AuditNames", StringUtils.join(configs.u, ","));
             content.put("LogConfiguration", map);
         } catch (IOException e) {
             LOG.error(e);

diff --git a/ui/src/pages/logs/index.tsx b/ui/src/pages/logs/index.tsx
@@ -56,7 +56,8 @@ export default function Logs(params: any) {
         <Typography style={{padding: '30px'}}>
             <Title>Log Configuration</Title>
             <Paragraph>
-                <p>Level: {LogConfiguration.VerboseNames}</p>
+                <p>Level: {LogConfiguration.Level}</p>
+                <p>Mode: {LogConfiguration.Mode}</p>
                 <p>Verbose Names:{LogConfiguration.VerboseNames}</p>
                 <p>Audit Names: {LogConfiguration.AuditNames}</p>
             </Paragraph>