hadoop-mapreduce-commits mailing list archives

Author: acmurthy
Date: Thu Oct 27 08:31:44 2011
New Revision: 1189670

URL: http://svn.apache.org/viewvc?rev=1189670&view=rev
Log:
MAPREDUCE-3279. Fixed TestJobHistoryParsing which assumed user name to be mapred all the time.
Contributed by Siddharth Seth.

Modified:
    hadoop/common/trunk/hadoop-mapreduce-project/CHANGES.txt
    hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-hs/src/test/java/org/apache/hadoop/mapreduce/v2/hs/TestJobHistoryParsing.java

Modified: hadoop/common/trunk/hadoop-mapreduce-project/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/common/trunk/hadoop-mapreduce-project/CHANGES.txt?rev=1189670&r1=1189669&r2=1189670&view=diff
==============================================================================
--- hadoop/common/trunk/hadoop-mapreduce-project/CHANGES.txt (original)
+++ hadoop/common/trunk/hadoop-mapreduce-project/CHANGES.txt Thu Oct 27 08:31:44 2011
@@ -1801,6 +1801,9 @@ Release 0.23.0 - Unreleased
     MAPREDUCE-3259. Added java.library.path of NodeManager to
     ContainerLocalizer in LinuxContainerExecutor. (Kihwal Lee via acmurthy) 
 
+    MAPREDUCE-3279. Fixed TestJobHistoryParsing which assumed user name to be
+    mapred all the time. (Siddharth Seth via acmurthy)
+
 Release 0.22.0 - Unreleased
 
   INCOMPATIBLE CHANGES

Modified: hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-hs/src/test/java/org/apache/hadoop/mapreduce/v2/hs/TestJobHistoryParsing.java
URL: http://svn.apache.org/viewvc/hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-hs/src/test/java/org/apache/hadoop/mapreduce/v2/hs/TestJobHistoryParsing.java?rev=1189670&r1=1189669&r2=1189670&view=diff
==============================================================================
--- hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-hs/src/test/java/org/apache/hadoop/mapreduce/v2/hs/TestJobHistoryParsing.java
(original)
+++ hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-hs/src/test/java/org/apache/hadoop/mapreduce/v2/hs/TestJobHistoryParsing.java
Thu Oct 27 08:31:44 2011
@@ -95,7 +95,8 @@ public class TestJobHistoryParsing {
     JobHistoryParser parser = new JobHistoryParser(in);
     JobInfo jobInfo = parser.parse();
 
-    Assert.assertEquals("Incorrect username ", "mapred", jobInfo.getUsername());
+    Assert.assertEquals("Incorrect username ", System.getProperty("user.name"),
+        jobInfo.getUsername());
     Assert.assertEquals("Incorrect jobName ", "test", jobInfo.getJobname());
     Assert.assertEquals("Incorrect queuename ", "default",
         jobInfo.getJobQueueName());
@@ -182,7 +183,7 @@ public class TestJobHistoryParsing {
         Integer.parseInt(jobSummaryElements.get("numMaps")));
     Assert.assertEquals("Mismatch in num reduce slots", 1,
         Integer.parseInt(jobSummaryElements.get("numReduces")));
-    Assert.assertEquals("User does not match", "mapred",
+    Assert.assertEquals("User does not match", System.getProperty("user.name"),
         jobSummaryElements.get("user"));
     Assert.assertEquals("Queue does not match", "default",
         jobSummaryElements.get("queue"));