MAPREDUCE-3112. Fixed recursive sourcing of HADOOP_OPTS environment

variable. (Eric Yang) git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/trunk@1178657 13f79535-47bb-0310-9956-ffa450edef68
sangameshs · Oct 4, 2011 · a48e6f2 · a48e6f2
1 parent 3edf357
commit a48e6f2
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/hadoop-common-project/hadoop-common/src/main/packages/templates/conf/hadoop-env.sh b/hadoop-common-project/hadoop-common/src/main/packages/templates/conf/hadoop-env.sh
@@ -41,7 +41,7 @@ done
 #export HADOOP_NAMENODE_INIT_HEAPSIZE=""
 
 # Extra Java runtime options.  Empty by default.
-export HADOOP_OPTS="-Djava.net.preferIPv4Stack=true $HADOOP_OPTS"
+export HADOOP_OPTS="-Djava.net.preferIPv4Stack=true $HADOOP_CLIENT_OPTS"
 
 # Command specific options appended to HADOOP_OPTS when specified
 export HADOOP_NAMENODE_OPTS="-Dsecurity.audit.logger=INFO,DRFAS -Dhdfs.audit.logger=INFO,DRFAAUDIT ${HADOOP_NAMENODE_OPTS}"

diff --git a/hadoop-mapreduce-project/CHANGES.txt b/hadoop-mapreduce-project/CHANGES.txt
@@ -1507,6 +1507,9 @@ Release 0.23.0 - Unreleased
     MAPREDUCE-2792. Replace usage of node ip-addresses with hostnames.
     (vinodkv via acmurthy) 
 
+    MAPREDUCE-3112. Fixed recursive sourcing of HADOOP_OPTS environment
+    variable. (Eric Yang)
+
 Release 0.22.0 - Unreleased
 
   INCOMPATIBLE CHANGES

diff --git a/...preduce-project/src/contrib/streaming/src/java/org/apache/hadoop/streaming/StreamJob.java b/...preduce-project/src/contrib/streaming/src/java/org/apache/hadoop/streaming/StreamJob.java
@@ -180,7 +180,9 @@ protected void init() {
 
   void preProcessArgs() {
     verbose_ = false;
-    addTaskEnvironment_ = "";
+    // Unset HADOOP_ROOT_LOGGER in case streaming job
+    // invokes additional hadoop commands.
+    addTaskEnvironment_ = "HADOOP_ROOT_LOGGER=";
   }
 
   void postProcessArgs() throws IOException {