[ 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