[ 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)