[ 
https://issues.apache.org/jira/browse/TEZ-2176?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Vasanth kumar RJ updated TEZ-2176:
----------------------------------
    Attachment: TEZ-2176.1.patch

[~sseth], Implemented your comments.
Changed most of the files to slf4j. Few files to log4j since it has dependency 
like "fatal" method.
In MRCounters.java, Check whether log method is required or not. I removed the 
logger printing implementation, because overriding method argument has 
commons-logging.
ShuffleManager  has both log4j and slf4j because a method requires slf4j and 
log4j for fatal.
Please review.

> Move all logging to slf4j
> -------------------------
>
>                 Key: TEZ-2176
>                 URL: https://issues.apache.org/jira/browse/TEZ-2176
>             Project: Apache Tez
>          Issue Type: Improvement
>            Reporter: Siddharth Seth
>            Assignee: Vasanth kumar RJ
>         Attachments: TEZ-2176.1.patch, TEZ-2176.patch
>
>
> SLF4J supports a more comprehensive set of APIs - MDC, Formatted strings.
> Also drop commons-logging from the dependency set.



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

Reply via email to