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

Hitesh Shah commented on TEZ-1521:
----------------------------------

Yes that makes sense.

I was looking to see if there was a way to simplify this code path. There is 
both handleRoutedTezEvents and the RouteEventTransition - in some cases we 
invoke the function directly and in some via the transition.

+1. 

> VertexDataMovementEventsGeneratedEvent may be logged twice in recovery log 
> ---------------------------------------------------------------------------
>
>                 Key: TEZ-1521
>                 URL: https://issues.apache.org/jira/browse/TEZ-1521
>             Project: Apache Tez
>          Issue Type: Sub-task
>            Reporter: Jeff Zhang
>            Assignee: Jeff Zhang
>            Priority: Critical
>         Attachments: TEZ-1521-1.patch
>
>
> The TezEvents may be added to pendingTaskEvents and route again later when 
> task is not scheduled. In this case, VertexDataMovementEventsGeneratedEvent 
> will been logged twice.



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

Reply via email to