You are viewing a plain text version of this content. The canonical link for it is here.
Posted to mapreduce-commits@hadoop.apache.org by dd...@apache.org on 2009/08/24 10:42:15 UTC

svn commit: r807123 - /hadoop/mapreduce/trunk/src/contrib/streaming/src/java/org/apache/hadoop/streaming/StreamJob.java

Author: ddas
Date: Mon Aug 24 08:42:15 2009
New Revision: 807123

URL: http://svn.apache.org/viewvc?rev=807123&view=rev
Log:
MAPREDUCE-767. Continuation of the patch that got committe earlier. This commit fixes a bug in StreamJob.java to do with handling -debug option.

Modified:
    hadoop/mapreduce/trunk/src/contrib/streaming/src/java/org/apache/hadoop/streaming/StreamJob.java

Modified: hadoop/mapreduce/trunk/src/contrib/streaming/src/java/org/apache/hadoop/streaming/StreamJob.java
URL: http://svn.apache.org/viewvc/hadoop/mapreduce/trunk/src/contrib/streaming/src/java/org/apache/hadoop/streaming/StreamJob.java?rev=807123&r1=807122&r2=807123&view=diff
==============================================================================
--- hadoop/mapreduce/trunk/src/contrib/streaming/src/java/org/apache/hadoop/streaming/StreamJob.java (original)
+++ hadoop/mapreduce/trunk/src/contrib/streaming/src/java/org/apache/hadoop/streaming/StreamJob.java Mon Aug 24 08:42:15 2009
@@ -249,7 +249,7 @@
     if (cmdLine != null){
       verbose_ =  cmdLine.hasOption("verbose");
       detailedUsage_ = cmdLine.hasOption("info");
-      debug_ = cmdLine.hasOption("-debug")? debug_ + 1 : debug_;
+      debug_ = cmdLine.hasOption("debug")? debug_ + 1 : debug_;
       
       String[] values = cmdLine.getOptionValues("input");
       if (values != null && values.length > 0) {
@@ -309,7 +309,7 @@
       if (null != jobconf && jobconf.length > 0){
         LOG.warn("-jobconf option is deprecated, please use -D instead.");
         for(String s : jobconf){
-          String[] parts = s.split("=",2);
+          String[] parts = s.split("=", 2);
           config_.set(parts[0], parts[1]);
         }
       }