[ https://issues.apache.org/jira/browse/ZOOKEEPER-2659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15823237#comment-15823237 ]
ASF GitHub Bot commented on ZOOKEEPER-2659: ------------------------------------------- Github user mikaelstaldal commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/148#discussion_r96145959 --- Diff: src/java/test/org/apache/zookeeper/server/quorum/QuorumPeerMainTest.java --- @@ -413,13 +418,18 @@ public void testBadPeerAddressInQuorum() throws Exception { ClientBase.setupTestEnv(); // setup the logger to capture all logs + LoggerContext loggerContext = (LoggerContext) LogManager.getContext(false); + Configuration logConfig = loggerContext.getConfiguration(); + Layout layout = - Logger.getRootLogger().getAppender("CONSOLE").getLayout(); + logConfig.getRootLogger().getAppenders().get("STDOUT").getLayout(); + String appenderName = "quorumAppender"; ByteArrayOutputStream os = new ByteArrayOutputStream(); - WriterAppender appender = new WriterAppender(layout, os); - appender.setThreshold(Level.WARN); - Logger qlogger = Logger.getLogger("org.apache.zookeeper.server.quorum"); - qlogger.addAppender(appender); + WriterAppender appender = WriterAppender.createAppender((StringLayout)layout, null, new OutputStreamWriter(os), appenderName, false, true); --- End diff -- It's probably a good idea to use the builder to create the appender: ``` WriterAppender.newBuilder().setLayout((StringLayout)layout).setTarget(new OutputStreamWriter(os)).setName(appenderName).build(); ``` > Use log4j2 as a logging framework as log4j 1.X is now deprecated > ---------------------------------------------------------------- > > Key: ZOOKEEPER-2659 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2659 > Project: ZooKeeper > Issue Type: Wish > Reporter: Pushkar Raste > Priority: Minor > Attachments: zk_log4j2_migration.patch > > > Zookeeper currently uses {{log4j 1.X}} as the default logging framework. > {{log4j 1.X}} is now deprecated http://logging.apache.org/log4j/1.2/ > This ticket is to track efforts to move zookeeper to {{log4j2}} -- This message was sent by Atlassian JIRA (v6.3.4#6332)