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

Siddharth Seth commented on TEZ-1816:
-------------------------------------

+1 with one minor change. The test needs to wait for all transitions.
{code}
+    dagWithCustomEdge.handle(new DAGEvent(dagWithCustomEdge.getID(), 
DAGEventType.DAG_START));
+    Assert.assertEquals(DAGState.FAILED, dagWithCustomEdge.getState());
{code}
should be
{code}
+    dagWithCustomEdge.handle(new DAGEvent(dagWithCustomEdge.getID(), 
DAGEventType.DAG_START));
dispatcher.await();
+    Assert.assertEquals(DAGState.FAILED, dagWithCustomEdge.getState());
{code}

> It is possible to receive START event when DAG is failed
> --------------------------------------------------------
>
>                 Key: TEZ-1816
>                 URL: https://issues.apache.org/jira/browse/TEZ-1816
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Jeff Zhang
>            Assignee: Jeff Zhang
>         Attachments: TEZ-1816.patch
>
>
> If DAG is failed in initialization, then it would goto failed. Then there 
> will be a following START event from client. 



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

Reply via email to