diff --git a/hadoop-common-project/hadoop-common/CHANGES.txt b/hadoop-common-project/hadoop-common/CHANGES.txt index fd40a89539..e09f45d499 100644 --- a/hadoop-common-project/hadoop-common/CHANGES.txt +++ b/hadoop-common-project/hadoop-common/CHANGES.txt @@ -370,7 +370,10 @@ Trunk (Unreleased) HADOOP-9481. Broken conditional logic with HADOOP_SNAPPY_LIBRARY. (Vadim Bondarev via atm) - OPTIMIZATIONS + HADOOP-9593. stack trace printed at ERROR for all yarn clients without + hadoop.home set (stevel) + + OPTIMIZATIONS HADOOP-7761. Improve the performance of raw comparisons. (todd) diff --git a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/Shell.java b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/Shell.java index 18ee182f53..75dc845f72 100644 --- a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/Shell.java +++ b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/Shell.java @@ -227,8 +227,10 @@ private static String checkHadoopHome() { home = homedir.getCanonicalPath(); } catch (IOException ioe) { - LOG.error("Failed to detect a valid hadoop home directory", ioe); - home = null; + if (LOG.isDebugEnabled()) { + LOG.debug("Failed to detect a valid hadoop home directory", ioe); + } + home = null; } return home; @@ -305,10 +307,13 @@ private static boolean isSetsidSupported() { shexec = new ShellCommandExecutor(args); shexec.execute(); } catch (IOException ioe) { - LOG.warn("setsid is not available on this machine. So not using it."); + LOG.debug("setsid is not available on this machine. So not using it."); setsidSupported = false; } finally { // handle the exit code - LOG.info("setsid exited with exit code " + shexec.getExitCode()); + if (LOG.isDebugEnabled()) { + LOG.debug("setsid exited with exit code " + + (shexec != null ? shexec.getExitCode() : "(null executor)")); + } } return setsidSupported; }