You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@whirr.apache.org by ab...@apache.org on 2012/08/24 01:24:14 UTC

svn commit: r1376771 - in /whirr/trunk: CHANGES.txt services/cdh/src/main/resources/functions/configure_cdh_hadoop.sh services/hadoop/src/main/resources/functions/configure_hadoop.sh

Author: abayer
Date: Thu Aug 23 23:24:13 2012
New Revision: 1376771

URL: http://svn.apache.org/viewvc?rev=1376771&view=rev
Log:
WHIRR-641. Race condition between collocated namenode and jobtracker start/init services. Contributed by Graham Gear.

Modified:
    whirr/trunk/CHANGES.txt
    whirr/trunk/services/cdh/src/main/resources/functions/configure_cdh_hadoop.sh
    whirr/trunk/services/hadoop/src/main/resources/functions/configure_hadoop.sh

Modified: whirr/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/whirr/trunk/CHANGES.txt?rev=1376771&r1=1376770&r2=1376771&view=diff
==============================================================================
--- whirr/trunk/CHANGES.txt (original)
+++ whirr/trunk/CHANGES.txt Thu Aug 23 23:24:13 2012
@@ -11,6 +11,9 @@ Trunk (unreleased changes)
     WHIRR-641. Improved parsing of config ->
     TemplateBuilderSpec. (Adrian Cole via abayer)
 
+    WHIRR-645. Race condition between collocated namenode and
+    jobtracker services. (Graham Gear via abayer)
+
 Release 0.8.0 - 2012-08-17
 
   NEW FEATURES

Modified: whirr/trunk/services/cdh/src/main/resources/functions/configure_cdh_hadoop.sh
URL: http://svn.apache.org/viewvc/whirr/trunk/services/cdh/src/main/resources/functions/configure_cdh_hadoop.sh?rev=1376771&r1=1376770&r2=1376771&view=diff
==============================================================================
--- whirr/trunk/services/cdh/src/main/resources/functions/configure_cdh_hadoop.sh (original)
+++ whirr/trunk/services/cdh/src/main/resources/functions/configure_cdh_hadoop.sh Thu Aug 23 23:24:13 2012
@@ -68,11 +68,12 @@ function configure_cdh_hadoop() {
   chgrp -R hadoop $HADOOP_LOG_DIR
   chmod -R g+w $HADOOP_LOG_DIR
 
+  if [ $(echo "$ROLES" | grep "hadoop-namenode" | wc -l) -gt 0 ]; then
+    start_namenode
+  fi
+  
   for role in $(echo "$ROLES" | tr "," "\n"); do
     case $role in
-    hadoop-namenode)
-      start_namenode
-      ;;
     hadoop-secondarynamenode)
       start_hadoop_daemon $HDFS_PACKAGE_PREFIX-secondarynamenode
       ;;

Modified: whirr/trunk/services/hadoop/src/main/resources/functions/configure_hadoop.sh
URL: http://svn.apache.org/viewvc/whirr/trunk/services/hadoop/src/main/resources/functions/configure_hadoop.sh?rev=1376771&r1=1376770&r2=1376771&view=diff
==============================================================================
--- whirr/trunk/services/hadoop/src/main/resources/functions/configure_hadoop.sh (original)
+++ whirr/trunk/services/hadoop/src/main/resources/functions/configure_hadoop.sh Thu Aug 23 23:24:13 2012
@@ -55,12 +55,13 @@ function configure_hadoop() {
   mkdir -p $(dirname $HADOOP_LOG_DIR)
   ln -s /data/hadoop/logs $HADOOP_LOG_DIR
   chown -R hadoop:hadoop $HADOOP_LOG_DIR
-  
+
+  if [ $(echo "$ROLES" | grep "hadoop-namenode" | wc -l) -gt 0 ]; then
+    start_namenode
+  fi
+
   for role in $(echo "$ROLES" | tr "," "\n"); do
     case $role in
-    hadoop-namenode)
-      start_namenode
-      ;;
     hadoop-secondarynamenode)
       start_hadoop_daemon secondarynamenode
       ;;