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

Bikas Saha commented on TEZ-1170:
---------------------------------

bq.  I think we need to fix at least the ROOT_INPUT_INITIALIZED state 
transitions which were changed by this patch.
Do you mean this ?
bq. The shutdown is invoked as soon as a KILL comes in (which could be via a 
user event). The Vertex processes the KILL and tries to shutdown the 
initializers - however, some initialization_DONE events may be in queue or 
generated if the initializer does not respect interrupts or wins a race between 
the interrupt and completing.
You mean the code in TerminateInitingVertexTransition?
If yes, then it makes sense.

> Simplify Vertex Initializing transition
> ---------------------------------------
>
>                 Key: TEZ-1170
>                 URL: https://issues.apache.org/jira/browse/TEZ-1170
>             Project: Apache Tez
>          Issue Type: Improvement
>            Reporter: Siddharth Seth
>            Assignee: Bikas Saha
>             Fix For: 0.5.0
>
>         Attachments: TEZ-1170.1.patch, TEZ-1170.2.patch
>
>
> After TEZ-1145 and 1151, a vertex should only need to stay in INITIALZING 
> state when it has an uninitialized edge, or when the parallelism is at -1 
> (not set yet). Waiting for all RootInputInitializers to complete should not 
> be required - as long as one of them sets parallelism (via a VertexManager).



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to