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

Bikas Saha commented on TEZ-2325:
---------------------------------

+1. 
Comments that can be fixed without further review unless there is some bug 
revealed due to them
Uncomment?
{code}+  @Test //(timeout = 5000){code}

The tests could get better coverage if we add cases for multiple events [0+ TA 
events & 0+ Other events] (includes 0 events overall).

> Route status update event directly to the attempt 
> --------------------------------------------------
>
>                 Key: TEZ-2325
>                 URL: https://issues.apache.org/jira/browse/TEZ-2325
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Bikas Saha
>            Assignee: Prakash Ramachandran
>         Attachments: TEZ-2325.1.patch
>
>
> Today, all events from the attempt heartbeat are routed to the vertex. then 
> the vertex routes (if any) status update events to the attempt. This is 
> unnecessary and potentially creates out of order scenarios. We could route 
> the status update events directly to attempts.



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

Reply via email to