hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Mit Desai (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (YARN-1833) TestRMAdminService Fails in trunk and branch-2 : Assert Fails due to different count of UserGroups for currentUser()
Date Wed, 26 Mar 2014 22:42:16 GMT

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

Mit Desai commented on YARN-1833:
---------------------------------

Changed the Summary of the JIRA to differentiate this issue with YARN-1883

> 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)

Mime
View raw message