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

Bikas Saha commented on TEZ-2011:
---------------------------------

bq. synchronized in onVertexManagerEventReceived, onRootVertexInitialized, 
onVertexStateUpdated, onVertexStarted added unintentionally
No. Added them to all public methods to make it consistently locked.

Thanks for the review!

> 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