Hadoop QA added a comment - 09/Aug/16 17:37 -1 overall Vote Subsystem Runtime Comment 0 reexec 0m 15s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 test4tests 0m 0s The patch appears to include 1 new or modified test files. +1 mvninstall 7m 2s trunk passed +1 compile 0m 18s trunk passed +1 checkstyle 0m 13s trunk passed +1 mvnsite 0m 22s trunk passed +1 mvneclipse 0m 13s trunk passed +1 findbugs 0m 29s trunk passed +1 javadoc 0m 13s trunk passed +1 mvninstall 0m 17s the patch passed +1 compile 0m 16s the patch passed +1 javac 0m 16s the patch passed -1 checkstyle 0m 11s hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-hs: The patch generated 1 new + 11 unchanged - 1 fixed = 12 total (was 12) +1 mvnsite 0m 20s the patch passed +1 mvneclipse 0m 11s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 findbugs 0m 34s the patch passed +1 javadoc 0m 11s the patch passed +1 unit 6m 47s hadoop-mapreduce-client-hs in the patch passed. +1 asflicense 0m 15s The patch does not generate ASF License warnings. 18m 45s Subsystem Report/Notes Docker Image:yetus/hadoop:9560f25 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12822836/MAPREDUCE-6750.patch JIRA Issue MAPREDUCE-6750 Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux 282aa73c5f76 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / 3cd386b Default Java 1.8.0_101 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/6667/artifact/patchprocess/diff-checkstyle-hadoop-mapreduce-project_hadoop-mapreduce-client_hadoop-mapreduce-client-hs.txt Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/6667/testReport/ modules C: hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-hs U: hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-hs Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/6667/console Powered by Apache Yetus 0.3.0 http://yetus.apache.org This message was automatically generated.

Varun Saxena added a comment - 09/Aug/16 17:55 +1. Will commit it.

Daryn Sharp added a comment - 09/Aug/16 17:56 +1 Thanks Kihwal.

Varun Saxena added a comment - 09/Aug/16 18:19 Committed to trunk, branch-2. Thanks Kihwal Lee for your contribution and Daryn Sharp for the review.

Hudson added a comment - 09/Aug/16 18:20 SUCCESS: Integrated in Hadoop-trunk-Commit #10245 (See https://builds.apache.org/job/Hadoop-trunk-Commit/10245/ ) MAPREDUCE-6750 . Fix TestHSAdminServer#testRefreshSuperUserGroups (Kihwal (varunsaxena: rev c4b77aed7e756dd463fd88141054ea0e3acd6396) hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-hs/src/test/java/org/apache/hadoop/mapreduce/v2/hs/server/TestHSAdminServer.java

Ray Chiang added a comment - 06/Oct/16 16:56 Just adding a note. Before this fix, the test breaks with the following error: java.lang.AssertionError: Unexpected exception thrown: org.apache.hadoop.security.authorize.AuthorizationException: User: superuser is not allowed to impersonate regularUser expected null, but was:<org.apache.hadoop.security.authorize.AuthorizationException: User: superuser is not allowed to impersonate regularUser> at org.junit.Assert.fail(Assert.java:88) at org.junit.Assert.failNotNull(Assert.java:664) at org.junit.Assert.assertNull(Assert.java:646) at org.apache.hadoop.mapreduce.v2.hs.server.TestHSAdminServer.testRefreshSuperUserGroups(TestHSAdminServer.java:236)