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

Karthik Kambatla commented on YARN-585:
---------------------------------------

Thanks for taking care of this, [~zjshen]. The fix looks mostly good. 

Minor comment: Sleep might be better in the following form.
{code}
for (int i = 0; i < 20 && state != SUBMITTED; i++) {
  Thread.sleep(100)
}
assertTrue("Didn't move to SUBMITTED to", state == SUBMITTED) 
{code}
                
> TestFairScheduler#testNotAllowSubmitApplication is broken due to YARN-514
> -------------------------------------------------------------------------
>
>                 Key: YARN-585
>                 URL: https://issues.apache.org/jira/browse/YARN-585
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Zhijie Shen
>            Assignee: Zhijie Shen
>         Attachments: YARN-585.1.patch
>
>
> TestFairScheduler#testNotAllowSubmitApplication is broken due to YARN-514. 
> See the discussions in YARN-514.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to