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