[ https://issues.apache.org/jira/browse/YARN-4807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15218824#comment-15218824 ]
Yufei Gu commented on YARN-4807: -------------------------------- Thanks [~sunilg] for review. Nice suggestion about SEC. I'll put them in my new patch. I think the "wait time per loop" is really matter. The patch tremendously reduce it from 500ms/200ms to 10ms. Is it possible to raise the changes of flaky tests if we reduce the length of timeout as [~templedf] suggested? And if we'd like to find an more accurate timeout, we could file another jira and do multiple round of Jenkins jobs for all related unit tests. > 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 > Attachments: YARN-4807.001.patch, YARN-4807.002.patch, > YARN-4807.003.patch, YARN-4807.004.patch > > > 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)