[ 
https://issues.apache.org/jira/browse/PIRK-11?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15390574#comment-15390574
 ] 

ASF GitHub Bot commented on PIRK-11:
------------------------------------

Github user smarthi commented on the issue:

    https://github.com/apache/incubator-pirk/pull/20
  
    Please review this (its huge :) )
    1. Added slf4j with log4j2 bindings
    2. Removed LogUtils.java
    3. Modified all of the code to use slf4j loggers
    4. Java 7 compatible code cleanup, removed redundant variables 


> Switch to SLF4J with log4j2 bindings
> ------------------------------------
>
>                 Key: PIRK-11
>                 URL: https://issues.apache.org/jira/browse/PIRK-11
>             Project: PIRK
>          Issue Type: Improvement
>            Reporter: Ellison Anne Williams
>            Assignee: Suneel Marthi
>
> Since log4j is EOL, we should upgrade to log4j2.
> Note: There has also been some discussion on the dev list on whether we 
> should also adopt Slf4j. When that decision is made, we can update the issue 
> or issue a new one. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to