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

Rohith Sharma K S commented on YARN-5988:
-----------------------------------------

bq. Is this really required ? as Resource manager is a much heavier class and 
to just test a small part is it required to ensure everything else is 
configured and working fine ? If its just one additional method at 
default/protected package visibility then i think it should be fine right ?
I am not fan of exposing method with annotation @VisibleForTesting unless that 
is really required. Here, there is option to simulate same test case refer 
{{TestRMAdminService#testServiceAclsRefreshWithLocalConfigurationProvider}}

> RM unable to start in secure setup
> ----------------------------------
>
>                 Key: YARN-5988
>                 URL: https://issues.apache.org/jira/browse/YARN-5988
>             Project: Hadoop YARN
>          Issue Type: Bug
>    Affects Versions: 2.9.0, 3.0.0-alpha1
>            Reporter: Ajith S
>            Assignee: Ajith S
>            Priority: Blocker
>         Attachments: YARN-5988.01.patch, YARN-5988.02.patch
>
>
> When CommonConfigurationKeysPublic.HADOOP_SECURITY_AUTHORIZATION=true
> RM is unable to start



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to