[
https://issues.apache.org/jira/browse/HDFS-9130?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Mingliang Liu updated HDFS-9130:
--------------------------------
Status: Patch Available (was: Open)
> Use GenericTestUtils#setLogLevel to set log4j or slf4j logger's level in unit
> tests
> -----------------------------------------------------------------------------------
>
> Key: HDFS-9130
> URL: https://issues.apache.org/jira/browse/HDFS-9130
> Project: Hadoop HDFS
> Issue Type: Task
> Reporter: Mingliang Liu
> Assignee: Mingliang Liu
> Attachments: HDFS-9130.000.patch
>
>
> Currently we use both commons-logging and slf4j in {{hadoop-hdfs}}. To change
> the change the logger level for dumping verbose debug information, there are
> many unit tests that just cast the LOG object to a {{Log4JLogger}} and call
> {{setLevel}} on that. e.g.
> {code}
> ((Log4JLogger)LOG).getLogger().setLevel(Level.ALL);
> {code}
> One problem of this hard-coded approach is that we need to update the casting
> code in test if we replace the log4j logger with slf4j. For example, as we're
> creating a separate jar for hdfs-client (see [HDFS-6200]) which uses only
> slf4j, we need to replace the log4j logger with slf4j logger, and to update
> the casting for changing logger's level in unit tests as well.
> Instead, we can use the {{GenericTestUtils#setLogLevel}} (brought in
> [HADOOP-11430]) method for both types of logger. This method internally
> figures out the right thing to do based on the log / logger type. e.g.
> {code}
> GenericTestUtils.setLogLevel(LOG, Level.ALL);
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
