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

Daniel Templeton commented on YARN-7289:
----------------------------------------

Not a big deal, but {{testApplicationLifetimeMonitorWorker()}} would more 
usually be named {{doTestApplicationLifetimeMonitor()}} or 
{{testApplicationLifetimeMonitorInternal()}}.

Given that you're only testing two schedulers, it might be clearer in 
{{testApplicationLifetimeMonitor()}} to just call 
{{testApplicationLifetimeMonitorWorker()}} twice instead of building the list 
and iterating over it.

> TestApplicationLifetimeMonitor.testApplicationLifetimeMonitor times out
> -----------------------------------------------------------------------
>
>                 Key: YARN-7289
>                 URL: https://issues.apache.org/jira/browse/YARN-7289
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: resourcemanager
>            Reporter: Miklos Szegedi
>            Assignee: Miklos Szegedi
>         Attachments: YARN-7289.000.patch, YARN-7289.001.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to