(amoro) branch master updated: [AMORO-3420] Fix info-level log message printing issue (#3432)

[jinsongzhou]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 65ab732df [AMORO-3420] Fix info-level log message printing issue 
(#3432)
65ab732df is described below

commit 65ab732dfddc9ae3a740c6249244a725152ca02e
Author: ZhouJinsong <zhoujinsong0...@163.com>
AuthorDate: Sat Feb 8 10:02:55 2025 +0800

    [AMORO-3420] Fix info-level log message printing issue (#3432)
    
    * Do not set log level in codes
    
    * Fix log4j2.xml format
---
 .../amoro/server/terminal/local/LocalSessionFactory.java      |  1 -
 amoro-ams/src/main/resources/log4j2.xml                       |  6 ++++--
 dist/src/main/amoro-bin/conf/log4j2.xml                       | 11 ++++++++---
 3 files changed, 12 insertions(+), 6 deletions(-)

diff --git 
a/amoro-ams/src/main/java/org/apache/amoro/server/terminal/local/LocalSessionFactory.java
 
b/amoro-ams/src/main/java/org/apache/amoro/server/terminal/local/LocalSessionFactory.java
index b5698dc1b..edbcd4515 100644
--- 
a/amoro-ams/src/main/java/org/apache/amoro/server/terminal/local/LocalSessionFactory.java
+++ 
b/amoro-ams/src/main/java/org/apache/amoro/server/terminal/local/LocalSessionFactory.java
@@ -149,7 +149,6 @@ public class LocalSessionFactory implements 
TerminalSessionFactory {
       }
 
       context = SparkSession.builder().config(sparkconf).getOrCreate();
-      context.sparkContext().setLogLevel("WARN");
     }
 
     return context;
diff --git a/amoro-ams/src/main/resources/log4j2.xml 
b/amoro-ams/src/main/resources/log4j2.xml
index 7602b1833..0a6c76df2 100644
--- a/amoro-ams/src/main/resources/log4j2.xml
+++ b/amoro-ams/src/main/resources/log4j2.xml
@@ -41,9 +41,11 @@
     <Loggers>
         <Root level="debug">
             <AppenderRef ref="consoleAppender" level="info"/>
-            <AppenderRef ref="consoleAppender" level="error"/>
         </Root>
-        <Logger name="iceberg.metric"  level="info">
+        <Logger name="org.apache.spark" additivity="false">
+            <AppenderRef ref="consoleAppender" level="warn"/>
+        </Logger>
+        <Logger name="iceberg.metric"  level="info" additivity="false">
             <AppenderRef ref="icebergRestMetricReporter"/>
         </Logger>
     </Loggers>
diff --git a/dist/src/main/amoro-bin/conf/log4j2.xml 
b/dist/src/main/amoro-bin/conf/log4j2.xml
index f31dffe50..82bfe4678 100644
--- a/dist/src/main/amoro-bin/conf/log4j2.xml
+++ b/dist/src/main/amoro-bin/conf/log4j2.xml
@@ -108,14 +108,19 @@
     </Appenders>
 
     <Loggers>
-        <logger name="amoro.metric"  level="info">
+        <logger name="amoro.metric"  level="info" additivity="false">
             <!-- iceberg rest metric reporter logs -->
-            <AppenderRef ref="amoroMetricReporter" />
+            <AppenderRef ref="amoroMetricReporter"/>
+        </logger>
+
+        <logger name="org.apache.spark"  level="warn" additivity="false">
+            <AppenderRef ref="infoFileAppender"/>
+            <AppenderRef ref="consoleAppender" level="${CONSOLE_LEVEL}"/>
         </logger>
 
         <Root level="debug">
             <!--By default, debug logs are not printed-->
-<!--            <AppenderRef ref="debugFileAppender" level="debug"/>-->
+            <!--<AppenderRef ref="debugFileAppender" level="debug"/>-->
             <AppenderRef ref="infoFileAppender" level="info"/>
             <AppenderRef ref="errorFileAppender" level="error"/>
             <!-- SET CONSOLE_LOG_LEVEL=info to print console logger in docker 
-->