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

Rajesh Balamohan commented on TEZ-2011:
---------------------------------------

lgtm. +1. 

-synchronized in onVertexManagerEventReceived, onRootVertexInitialized, 
onVertexStateUpdated, onVertexStarted added unintentionally?
-plz remove unused import before committing


> InputReadyVertexManager not resilient to updates in parallelism
> ---------------------------------------------------------------
>
>                 Key: TEZ-2011
>                 URL: https://issues.apache.org/jira/browse/TEZ-2011
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Bikas Saha
>            Assignee: Bikas Saha
>         Attachments: TEZ-2011.1.patch
>
>
> It sets up its internal book-keeping upon onVertexStarted() but that 
> book-keeping can be wrong if its parallelism changes after that. It needs to 
> subscribe to Vertex configured event for its source vertices to make sure 
> that all changes have been made before initializing its internal state.



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

Reply via email to