Hadoop Common
  1. Hadoop Common
  2. HADOOP-8789

Tests setLevel(Level.OFF) should be Level.ERROR

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.1-alpha
    • Fix Version/s: 2.0.3-alpha, 0.23.5
    • Component/s: test
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      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.

      1. hdfs-3911.txt
        3 kB
        Andy Isaacson

        Activity

        No work has yet been logged on this issue.

          People

          • Assignee:
            Andy Isaacson
            Reporter:
            Andy Isaacson
          • Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development