Browse Source

MAPREDUCE-5621. mr-jobhistory-daemon.sh doesn't have to execute mkdir and chown all the time. Contributed by Shinichi Yamashita.

(cherry picked from commit ea3dba37cd81a238ffcca4bb41dd69089187bc11)
Junping Du 7 years ago
parent
commit
28be69ae6d
1 changed files with 4 additions and 2 deletions
  1. 4 2
      hadoop-mapreduce-project/bin/mr-jobhistory-daemon.sh

+ 4 - 2
hadoop-mapreduce-project/bin/mr-jobhistory-daemon.sh

@@ -78,8 +78,10 @@ if [ -f "${HADOOP_CONF_DIR}/mapred-env.sh" ]; then
   . "${HADOOP_CONF_DIR}/mapred-env.sh"
   . "${HADOOP_CONF_DIR}/mapred-env.sh"
 fi
 fi
 
 
-mkdir -p "$HADOOP_MAPRED_LOG_DIR"
-chown $HADOOP_MAPRED_IDENT_STRING $HADOOP_MAPRED_LOG_DIR
+if [ ! -w "$HADOOP_MAPRED_LOG_DIR" ]; then
+  mkdir -p "$HADOOP_MAPRED_LOG_DIR"
+  chown "$HADOOP_MAPRED_IDENT_STRING" "$HADOOP_MAPRED_LOG_DIR"
+fi
 
 
 if [ "$HADOOP_MAPRED_PID_DIR" = "" ]; then
 if [ "$HADOOP_MAPRED_PID_DIR" = "" ]; then
   HADOOP_MAPRED_PID_DIR=/tmp
   HADOOP_MAPRED_PID_DIR=/tmp