You are viewing a plain text version of this content. The canonical link for it is here.
Posted to yarn-issues@hadoop.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2014/04/11 22:40:16 UTC

[jira] [Commented] (YARN-1833) TestRMAdminService Fails in trunk and branch-2 : Assert Fails due to different count of UserGroups for currentUser()

    [ https://issues.apache.org/jira/browse/YARN-1833?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13967074#comment-13967074 ] 

Hudson commented on YARN-1833:
------------------------------

SUCCESS: Integrated in Hadoop-trunk-Commit #5506 (See [https://builds.apache.org/job/Hadoop-trunk-Commit/5506/])
YARN-1833. Relocating CHANGES.txt entry in trunk and branch-2 to the right location. (vinodkv: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1586769)
* /hadoop/common/branches/branch-2/hadoop-yarn-project/CHANGES.txt
* /hadoop/common/trunk/hadoop-yarn-project/CHANGES.txt


> TestRMAdminService Fails in trunk and branch-2 : Assert Fails due to different count of UserGroups for currentUser()
> --------------------------------------------------------------------------------------------------------------------
>
>                 Key: YARN-1833
>                 URL: https://issues.apache.org/jira/browse/YARN-1833
>             Project: Hadoop YARN
>          Issue Type: Bug
>    Affects Versions: 3.0.0, 2.4.0
>            Reporter: Mit Desai
>            Assignee: Mit Desai
>              Labels: Test
>             Fix For: 3.0.0, 2.4.0, 2.5.0
>
>         Attachments: YARN-1833-v2.patch, YARN-1833.patch
>
>
> In the test testRefreshUserToGroupsMappingsWithFileSystemBasedConfigurationProvider, the following assert is not needed.
> {code}
> Assert.assertTrue(groupWithInit.size() != groupBefore.size());
> {code}
> As the assert takes the default groups for groupWithInit (which in my case are users, sshusers and wheel), it fails as the size of both groupWithInit and groupBefore are same.
> I do not think we need to have this assert here. Moreover we are also checking that the groupInit does not have the userGroups that are in the groupBefore so removing the assert may not be harmful.



--
This message was sent by Atlassian JIRA
(v6.2#6252)