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 am...@apache.org on 2010/09/02 10:10:20 UTC

svn commit: r991846 - in /hadoop/mapreduce/trunk: CHANGES.txt src/contrib/streaming/src/test/org/apache/hadoop/streaming/TestFileArgs.java

Author: amareshwari
Date: Thu Sep  2 08:10:19 2010
New Revision: 991846

URL: http://svn.apache.org/viewvc?rev=991846&view=rev
Log:
MAPREDUCE-1375. Fixes flaky test TestFileArgs. Contributed by Todd Lipcon

Modified:
    hadoop/mapreduce/trunk/CHANGES.txt
    hadoop/mapreduce/trunk/src/contrib/streaming/src/test/org/apache/hadoop/streaming/TestFileArgs.java

Modified: hadoop/mapreduce/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/mapreduce/trunk/CHANGES.txt?rev=991846&r1=991845&r2=991846&view=diff
==============================================================================
--- hadoop/mapreduce/trunk/CHANGES.txt (original)
+++ hadoop/mapreduce/trunk/CHANGES.txt Thu Sep  2 08:10:19 2010
@@ -268,6 +268,9 @@ Trunk (unreleased changes)
     MAPREDUCE-2021. Fixes duplicate hostnames in CombineFileInputFormat's
     split locations. (amareshwari)
 
+    MAPREDUCE-1375. Fixes flaky test TestFileArgs. (Todd Lipcon via
+    amareshwari)
+
 Release 0.21.0 - Unreleased
 
   INCOMPATIBLE CHANGES

Modified: hadoop/mapreduce/trunk/src/contrib/streaming/src/test/org/apache/hadoop/streaming/TestFileArgs.java
URL: http://svn.apache.org/viewvc/hadoop/mapreduce/trunk/src/contrib/streaming/src/test/org/apache/hadoop/streaming/TestFileArgs.java?rev=991846&r1=991845&r2=991846&view=diff
==============================================================================
--- hadoop/mapreduce/trunk/src/contrib/streaming/src/test/org/apache/hadoop/streaming/TestFileArgs.java (original)
+++ hadoop/mapreduce/trunk/src/contrib/streaming/src/test/org/apache/hadoop/streaming/TestFileArgs.java Thu Sep  2 08:10:19 2010
@@ -72,6 +72,10 @@ public class TestFileArgs extends TestSt
     DataOutputStream dos = localFs.create(new Path("sidefile"));
     dos.write("hello world\n".getBytes("UTF-8"));
     dos.close();
+
+    // Since ls doesn't read stdin, we don't want to write anything
+    // to it, or else we risk Broken Pipe exceptions.
+    input = "";
   }
 
   @After