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

Íñigo Goiri commented on YARN-6553:
-----------------------------------

Thanks [~BilwaST] for the patch.
* Please fix the checkstyles that are fixable.
* TestableFederationClientInterceptor#74, let's log the possible exception.

> Replace MockResourceManagerFacade with MockRM for AMRMProxy/Router tests
> ------------------------------------------------------------------------
>
>                 Key: YARN-6553
>                 URL: https://issues.apache.org/jira/browse/YARN-6553
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Giovanni Matteo Fumarola
>            Assignee: Bilwa S T
>            Priority: Major
>         Attachments: YARN-6553.001.patch
>
>
> Currently the AMRMProxy and Router tests use the 
> {{MockResourceManagerFacade}}. This jira proposes replacing it with 
> {{MockRM}} as is done in majority of the tests.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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

Reply via email to