Xiang Yang has posted comments on this change. ( http://gerrit.cloudera.org:8080/21293 )
Change subject: IMPALA-12999: Add log4j.properties to the DEB/RPM packages ...................................................................... Patch Set 1: > Patch Set 1: > > > Miraculously, I can see the stacktrace without the 'conf/log4j.properties' > > file. > > Maybe log4j.properties from some jars is used. You can add the following > codes to print it: > > diff --git a/fe/src/main/java/org/apache/impala/util/GlogAppender.java > b/fe/src/main/java/org/apache/impala/util/GlogAppender.java > index 82c6c79cf..9ed49e45b 100644 > --- a/fe/src/main/java/org/apache/impala/util/GlogAppender.java > +++ b/fe/src/main/java/org/apache/impala/util/GlogAppender.java > @@ -115,6 +115,8 @@ public class GlogAppender extends AppenderSkeleton { > */ > public static void Install(TLogLevel impalaLogLevel, TLogLevel > otherLogLevel) > throws InternalException { > + System.out.println("log4j.xml: " + > Logger.getRootLogger().getClass().getResource("/log4j.xml")); > + System.out.println("log4j.properties: " + > Logger.getRootLogger().getClass().getResource("/log4j.properties")); > Properties properties = new Properties(); > properties.setProperty("log4j.appender.glog", > GlogAppender.class.getName()); Got it, I tested it in the dev env and the path 'fe/src/test/resources' was added in the CLASSPATH. -- To view, visit http://gerrit.cloudera.org:8080/21293 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iee0b9699ef313aa8e94bd351fa51fad3ea0cdf57 Gerrit-Change-Number: 21293 Gerrit-PatchSet: 1 Gerrit-Owner: Quanlong Huang <huangquanl...@gmail.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com> Gerrit-Reviewer: Xiang Yang <yx91...@126.com> Gerrit-Reviewer: Zihao Ye <eyiz...@163.com> Gerrit-Reviewer: Zoltan Borok-Nagy <borokna...@cloudera.com> Gerrit-Comment-Date: Sun, 14 Apr 2024 01:14:25 +0000 Gerrit-HasComments: No