[ 
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)

Reply via email to