hadoop-mapreduce-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From omal...@apache.org
Subject svn commit: r1079254 - in /hadoop/mapreduce/branches/yahoo-merge/src/test/mapred/org/apache/hadoop/mapred: TestJobQueueInformation.java TestSetupAndCleanupFailure.java
Date Tue, 08 Mar 2011 06:00:47 GMT
Author: omalley
Date: Tue Mar  8 06:00:47 2011
New Revision: 1079254

URL: http://svn.apache.org/viewvc?rev=1079254&view=rev
Log:
commit 8ba61c1eb9fec1f56a6acecc952353d2afd4a8bb
Author: Greg Roelofs <roelofs@yahoo-inc.com>
Date:   Tue Jan 25 15:28:37 2011 -0800

    Added assert messages for two "mixed-mode" unit tests (both formerly failed
    on same assert in both uber and non-uber mode [at least, according to my
    notes]; now uber still does but non-uber times out.  Since _probably_ not
    uber-specific problem, leaving actual fix(es) for later.

Modified:
    hadoop/mapreduce/branches/yahoo-merge/src/test/mapred/org/apache/hadoop/mapred/TestJobQueueInformation.java
    hadoop/mapreduce/branches/yahoo-merge/src/test/mapred/org/apache/hadoop/mapred/TestSetupAndCleanupFailure.java

Modified: hadoop/mapreduce/branches/yahoo-merge/src/test/mapred/org/apache/hadoop/mapred/TestJobQueueInformation.java
URL: http://svn.apache.org/viewvc/hadoop/mapreduce/branches/yahoo-merge/src/test/mapred/org/apache/hadoop/mapred/TestJobQueueInformation.java?rev=1079254&r1=1079253&r2=1079254&view=diff
==============================================================================
--- hadoop/mapreduce/branches/yahoo-merge/src/test/mapred/org/apache/hadoop/mapred/TestJobQueueInformation.java
(original)
+++ hadoop/mapreduce/branches/yahoo-merge/src/test/mapred/org/apache/hadoop/mapred/TestJobQueueInformation.java
Tue Mar  8 06:00:47 2011
@@ -133,7 +133,8 @@ public class TestJobQueueInformation ext
       assertEquals(expectedQueueInfo, queueInfo.getSchedulingInfo());
       numberOfJobs += jobStatusList.length;
       for (JobStatus status : jobStatusList) {
-        assertEquals(JOB_SCHEDULING_INFO, status.getSchedulingInfo());
+        assertEquals("Custom JobStatus scheduling info not found.",
+                     JOB_SCHEDULING_INFO, status.getSchedulingInfo());
       }
     }
     assertEquals(1, numberOfJobs);

Modified: hadoop/mapreduce/branches/yahoo-merge/src/test/mapred/org/apache/hadoop/mapred/TestSetupAndCleanupFailure.java
URL: http://svn.apache.org/viewvc/hadoop/mapreduce/branches/yahoo-merge/src/test/mapred/org/apache/hadoop/mapred/TestSetupAndCleanupFailure.java?rev=1079254&r1=1079253&r2=1079254&view=diff
==============================================================================
--- hadoop/mapreduce/branches/yahoo-merge/src/test/mapred/org/apache/hadoop/mapred/TestSetupAndCleanupFailure.java
(original)
+++ hadoop/mapreduce/branches/yahoo-merge/src/test/mapred/org/apache/hadoop/mapred/TestSetupAndCleanupFailure.java
Tue Mar  8 06:00:47 2011
@@ -115,7 +115,7 @@ public class TestSetupAndCleanupFailure 
     RunningJob job = UtilsForTests.runJob(jobConf, inDir, outDir);
     // wait for the job to finish.
     job.waitForCompletion();
-    assertEquals(JobStatus.FAILED, job.getJobState());
+    assertEquals("job didn't fail!", JobStatus.FAILED, job.getJobState());
   }
   
   // launch job with CommitterWithLongSetupAndCleanup as committer



Mime
View raw message