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

Rajesh Balamohan commented on TEZ-2020:
---------------------------------------

lgtm. +1.

Asked about that test explicitly as the user needs to be be aware of 
auto-reduce parallelism in such cases. Should auto-reduce parallelism be 
disabled in such cases by framework itself?

> For 1-1 edge vertex configured event may be sent incorrectly
> ------------------------------------------------------------
>
>                 Key: TEZ-2020
>                 URL: https://issues.apache.org/jira/browse/TEZ-2020
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Bikas Saha
>            Assignee: Bikas Saha
>         Attachments: TEZ-2020.1.patch, synthetic_job.png
>
>
> For a fully configured 1-1 vertex configured event will be sent after init 
> because the vertex manager does not signal intent to reconfigure. However a 
> change in parallelism upstream on the 1-1 edge can cause this vertex's 
> parallelism to be updated accordingly. This breaks the guarantee that 
> configured should be sent after vertex is fully configured. This is because 
> update of parallelism on 1-1 edge has until now been hard-coded in the Vertex 
> code. This can be removed now with the InputReadyVertexManager handling the 
> 1-1 edge and the addition of vertex state updates. Not only will this fix the 
> issue but also remove the special handling for 1-1 edges in the AM which 
> should not be there for cleanliness.



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

Reply via email to