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

Rajesh Balamohan updated TEZ-1610:
----------------------------------
    Attachment: TEZ-1610.5.patch

Thanks [~sseth] for the review.  Attaching the patch addressing the review 
comments.
- startTime should be final in Shuffle (Done)
- can updateEventReceivedTime() just be called when handleEvent / 
processDataMovementEvent is invoked. (Done)
- is this check required ? "if (lastEventReceived.getValue() < relativeTime)" 
    (Added it as defensive check.  Removed it in latest patch).

Will commit once the rc1 vote is over .

> additional task counters for fetchers
> -------------------------------------
>
>                 Key: TEZ-1610
>                 URL: https://issues.apache.org/jira/browse/TEZ-1610
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Rajesh Balamohan
>            Assignee: Rajesh Balamohan
>         Attachments: TEZ-1610.1.patch, TEZ-1610.2.patch, TEZ-1610.4.patch, 
> TEZ-1610.5.patch
>
>
> - ShuffleFinishTime (per source)
> - Merge time (depending on broadcast/scatter-gather shuffle)
> This would be helpful in determining when shuffle started/ended for different 
> sources in a task.



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

Reply via email to