You are viewing a plain text version of this content. The canonical link for it is here.

Posted to common-commits@hadoop.apache.org by om...@apache.org on 2008/10/26 07:45:39 UTC

svn commit: r707943 - in /hadoop/core/trunk: CHANGES.txt bin/slaves.sh

Author: omalley
Date: Sat Oct 25 23:45:38 2008
New Revision: 707943

URL: http://svn.apache.org/viewvc?rev=707943&view=rev
Log:
HADOOP-4454. Allow # comments in slaves file. (Rama Ramasamy via omalley)

Modified:
    hadoop/core/trunk/CHANGES.txt
    hadoop/core/trunk/bin/slaves.sh

Modified: hadoop/core/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/core/trunk/CHANGES.txt?rev=707943&r1=707942&r2=707943&view=diff
==============================================================================
--- hadoop/core/trunk/CHANGES.txt (original)
+++ hadoop/core/trunk/CHANGES.txt Sat Oct 25 23:45:38 2008
@@ -56,6 +56,8 @@
 
     HADOOP-4510. Make getTaskOutputPath public. (Chris Wensel via omalley)
 
+    HADOOP-4454. Allow # comments in slaves file. (Rama Ramasamy via omalley)
+
   OPTIMIZATIONS
 
   BUG FIXES

Modified: hadoop/core/trunk/bin/slaves.sh
URL: http://svn.apache.org/viewvc/hadoop/core/trunk/bin/slaves.sh?rev=707943&r1=707942&r2=707943&view=diff
==============================================================================
--- hadoop/core/trunk/bin/slaves.sh (original)
+++ hadoop/core/trunk/bin/slaves.sh Sat Oct 25 23:45:38 2008
@@ -57,7 +57,7 @@
   fi
 fi
 
-for slave in `cat "$HOSTLIST"`; do
+for slave in `cat "$HOSTLIST"|sed  "s/#.*$//;/^$/d"`; do
  ssh $HADOOP_SSH_OPTS $slave $"${@// /\\ }" \
    2>&1 | sed "s/^/$slave: /" &
  if [ "$HADOOP_SLAVE_SLEEP" != "" ]; then