[HADOOP-8789] Tests setLevel(Level.OFF) should be Level.ERROR

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.0.1-alpha
• Fix Version/s: 2.0.3-alpha, 0.23.5
• Component/s: test
• Hadoop Flags: Reviewed
• Assignee: Andy Isaacson
• Reporter: Andy Isaacson
• Created: 10/Sep/12 19:04
• Updated: 06/Feb/13 17:05
• Resolved: 11/Sep/12 17:58

Description

Multiple tests have code like

((Log4JLogger)LogFactory.getLog(FSNamesystem.class)).getLogger().setLevel(Level.OFF);

Completely disabling logs from given classes with Level.OFF is a bad idea and makes debugging other test failures, especially intermittent test failures like HDFS-3664, difficult. Instead the code should use Level.ERROR to reduce verbosity.