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 2011/03/04 05:13:43 UTC

svn commit: r1077426 - /hadoop/common/branches/branch-0.20-security-patches/src/contrib/gridmix/src/test/org/apache/hadoop/mapred/gridmix/TestUserResolve.java

Author: omalley
Date: Fri Mar  4 04:13:43 2011
New Revision: 1077426

URL: http://svn.apache.org/viewvc?rev=1077426&view=rev
Log:
commit dc2d076ccf3e04c5aabafdfec149b4f94278e039
Author: Devaraj Das <dd...@yahoo-inc.com>
Date:   Sat Apr 24 17:09:44 2010 -0700

    HADOOP:6670 from https://issues.apache.org/jira/secure/attachment/12442760/6670-gridmix-test.bugfix.patch
    
    +++ b/YAHOO-CHANGES.txt
    +    HADOOP-6670. Fixes a testcase issue introduced by the earlier commit
    +    of the HADOOP-6670 patch. (ddas)
    +

Modified:
    hadoop/common/branches/branch-0.20-security-patches/src/contrib/gridmix/src/test/org/apache/hadoop/mapred/gridmix/TestUserResolve.java

Modified: hadoop/common/branches/branch-0.20-security-patches/src/contrib/gridmix/src/test/org/apache/hadoop/mapred/gridmix/TestUserResolve.java
URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-security-patches/src/contrib/gridmix/src/test/org/apache/hadoop/mapred/gridmix/TestUserResolve.java?rev=1077426&r1=1077425&r2=1077426&view=diff
==============================================================================
--- hadoop/common/branches/branch-0.20-security-patches/src/contrib/gridmix/src/test/org/apache/hadoop/mapred/gridmix/TestUserResolve.java (original)
+++ hadoop/common/branches/branch-0.20-security-patches/src/contrib/gridmix/src/test/org/apache/hadoop/mapred/gridmix/TestUserResolve.java Fri Mar  4 04:13:43 2011
@@ -72,13 +72,15 @@ public class TestUserResolve {
     assertTrue("User list required for RoundRobinUserResolver", fail);
 
     rslv.setTargetUsers(new URI(userlist.toString()), conf);
-    assertEquals("user0", rslv.getTargetUgi(UserGroupInformation.createRemoteUser("hfre0")).getUserName());
+    UserGroupInformation ugi1;
+    assertEquals("user0", 
+        rslv.getTargetUgi((ugi1 = 
+          UserGroupInformation.createRemoteUser("hfre0"))).getUserName());
     assertEquals("user1", rslv.getTargetUgi(UserGroupInformation.createRemoteUser("hfre1")).getUserName());
     assertEquals("user2", rslv.getTargetUgi(UserGroupInformation.createRemoteUser("hfre2")).getUserName());
-    assertEquals("user0", rslv.getTargetUgi(UserGroupInformation.createRemoteUser("hfre0")).getUserName());
+    assertEquals("user0", rslv.getTargetUgi(ugi1).getUserName());
     assertEquals("user3", rslv.getTargetUgi(UserGroupInformation.createRemoteUser("hfre3")).getUserName());
-    assertEquals("user0", rslv.getTargetUgi(UserGroupInformation.createRemoteUser("hfre0")).getUserName());
-    assertEquals("user0", rslv.getTargetUgi(UserGroupInformation.createRemoteUser("hfre4")).getUserName());
+    assertEquals("user0", rslv.getTargetUgi(ugi1).getUserName());
   }
 
   @Test