virajjasani commented on code in PR #5315: URL: https://github.com/apache/hadoop/pull/5315#discussion_r1089254542
########## hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/DataNode.java: ########## @@ -360,7 +358,8 @@ public class DataNode extends ReconfigurableBase FS_GETSPACEUSED_JITTER_KEY, FS_GETSPACEUSED_CLASSNAME)); - public static final Log METRICS_LOG = LogFactory.getLog("DataNodeMetricsLog"); + public static final org.apache.log4j.Logger METRICS_LOG = + org.apache.log4j.Logger.getLogger("DataNodeMetricsLog"); Review Comment: Unfortunately we already have bunch of direct log4j references all over. Replacing them with slf4j as much as possible would be part of my next set of sub-tasks (parent Jira HADOOP-16206). ########## hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/IOUtils.java: ########## @@ -256,7 +255,7 @@ public static void skipFully(InputStream in, long len) throws IOException { * instead */ @Deprecated - public static void cleanup(Log log, java.io.Closeable... closeables) { + public static void cleanup(Logger log, java.io.Closeable... closeables) { Review Comment: Yes that's valid. I am fine with any of these options: 1. We remove this method with this PR 2. We remove this method with a sub-task 3. We let it be here for a while and that would anyways mandate downstreamers to migrate to slf4j for using this utility -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org