dmvk commented on a change in pull request #18846:
URL: https://github.com/apache/flink/pull/18846#discussion_r810876705
##########
File path: tools/ci/log4j.properties
##########
@@ -37,15 +37,33 @@ appender.file.fileName =
${sys:log.dir}/mvn-${sys:mvn.forkNumber}.log
appender.file.layout.type = PatternLayout
appender.file.layout.pattern = %d{HH:mm:ss,SSS} [%20t] %-5p %-60c %x - %m%n
+# -----------------------------------------------------------------------------
+# ZooKeeper-related logs should end up here
+# -----------------------------------------------------------------------------
+appender.zk.name = ZooKeeperAppender
+appender.zk.type = FILE
+appender.zk.fileName = ${sys:log.dir}/zookeeper-${sys:mvn.forkNumber}.log
+appender.zk.layout.type = PatternLayout
+appender.zk.layout.pattern = %d{HH:mm:ss,SSS} [%20t] %-5p %-60c %x - %m%n
+
# suppress the irrelevant (wrong) warnings from the netty channel handler
logger.netty.name = org.jboss.netty.channel.DefaultChannelPipeline
logger.netty.level = ERROR
+
+# collecting ZooKeeper-related logs in separate log file
logger.zookeeper.name = org.apache.zookeeper
Review comment:
Would this push both client & server logs into a new file? Wouldn't that
be confusing to read?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
