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

sandflee commented on YARN-5375:
--------------------------------

thanks [~rohithsharma],   even if we using the order rmDispatcher.await --> 
stateStoreDispatcher --> rmDispatcher , there are still very very little change 
that events are not processed in stateStoreDispatcher or rmDispatcher, agree? 
 I prefer a accurate way to really drain all events if MockRM#drainEvents 
returned. based on this we could use drainEvents to replace sleep-wait way in 
MockRM#waitForState, this may help to reduce test time, thought?

> 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, 
> YARN-5375.04.patch, YARN-5375.05.patch, YARN-5375.06.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]

Reply via email to