You are viewing a plain text version of this content. The canonical link for it is here.

Posted to yarn-issues@hadoop.apache.org by "Sunil G (JIRA)" <ji...@apache.org> on 2016/03/13 11:05:33 UTC

[jira] [Commented] (YARN-4807) MockAM#waitForState sleep duration is too long

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