tez-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bikas Saha (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (TEZ-2015) VertexImpl.doneReconfiguringVertex() should check other criteria before sending notification
Date Tue, 03 Feb 2015 00:41:35 GMT

     [ https://issues.apache.org/jira/browse/TEZ-2015?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Bikas Saha updated TEZ-2015:
----------------------------
    Attachment: TEZ-2015.1.patch

Attaching fix to check for canInit() that checks if the vertex is completely configured or
not before sending the event. This is similar to other places that can result in sending this
signal. Test added to cover the condition. [~rajesh.balamohan] This removes the comment for
TEZ-2015 that was added in the patch for TEZ-2011. Please review.

> VertexImpl.doneReconfiguringVertex() should check other criteria before sending notification
> --------------------------------------------------------------------------------------------
>
>                 Key: TEZ-2015
>                 URL: https://issues.apache.org/jira/browse/TEZ-2015
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Bikas Saha
>            Assignee: Bikas Saha
>         Attachments: TEZ-2015.1.patch
>
>
> The vertex manager may have done what it wanted e.g. set parallelism but other items
like input edge may still not be defined. So state change might be sent incorrectly.



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

Mime
View raw message