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

Sunil G commented on YARN-4807:
-------------------------------

As I see, {{MockRM#waitForState}} is also considering 500ms to 1sec in various 
methods. I think that can be made smaller to 200ms. Recently myslef and 
[~rohithsharma] had worked on few random test cases failures in CS and all were 
fixed when we properly made use of {{waitForState}} with correct timeout. I 
think 200ms is more than enough, and we can remove MockAM method and 
consolidate all into MockRM. thoughts?

> MockAM#waitForState sleep duration is too long
> ----------------------------------------------
>
>                 Key: YARN-4807
>                 URL: https://issues.apache.org/jira/browse/YARN-4807
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>    Affects Versions: 2.8.0
>            Reporter: Karthik Kambatla
>            Assignee: Yufei Gu
>              Labels: newbie
>
> MockAM#waitForState sleep duration (500 ms) is too long. Also, there is 
> significant duplication with MockRM#waitForState.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to