[
https://issues.apache.org/jira/browse/YARN-5375?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15388972#comment-15388972
]
Sunil G commented on YARN-5375:
-------------------------------
Hi [~sandflee]
Thanks for the effort. Its a good initiative.
This patch exposes a new api in *RMStateStore* to set dispatcher. I think its
better and clean to have a mock class extended from RMStateStore, and override
to set dispatcher. {{YARN-5043}} has some done a similar effort for state
store. I think there is bit of duplicate effort only for statestore changes
with YARN-5043. I suggest you can check that point. cc/[~hex108]
> invoke MockRM#drainEvents implicitly in MockRM methods to reduce test failures
> ------------------------------------------------------------------------------
>
> Key: YARN-5375
> URL: https://issues.apache.org/jira/browse/YARN-5375
> Project: Hadoop YARN
> Issue Type: Improvement
> Reporter: sandflee
> Assignee: sandflee
> Attachments: YARN-5375.01.patch, YARN-5375.03.patch
>
>
> seen many test failures related to RMApp/RMAppattempt comes to some state but
> some event are not processed in rm event queue or scheduler event queue,
> cause test failure, seems we could implicitly invokes drainEvents(should also
> drain sheduler event) in some mockRM method like waitForState
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
