[
https://issues.apache.org/jira/browse/YARN-6411?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15949230#comment-15949230
]
Jason Lowe commented on YARN-6411:
----------------------------------
Thanks for the patch! Looks good overall. It would be nice to cleanup the
checkstyle issues of now unused Dispatcher/DrainDispatcher imports and eclipsed
fields.
> Clean up the overwrite of createDispatcher() in subclass of MockRM
> ------------------------------------------------------------------
>
> Key: YARN-6411
> URL: https://issues.apache.org/jira/browse/YARN-6411
> Project: Hadoop YARN
> Issue Type: Task
> Components: resourcemanager
> Affects Versions: 2.9.0, 3.0.0-alpha2
> Reporter: Yufei Gu
> Assignee: Yufei Gu
> Priority: Minor
> Attachments: YARN-6411.001.patch
>
>
> MockRM creates a object of {{DrainDispatcher}} in YARN-3102. We don't need to
> do the same thing in its subclasses.
> {code}
> @Override
> protected Dispatcher createDispatcher() {
> return new DrainDispatcher();
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
