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

Jonathan Eagles commented on TEZ-3607:
--------------------------------------

[~kshukla], could allEventsReceived also return true too early in the case of 
empty partitions? In addition what bug will surface in this case. Will there be 
a hang or will we try to shutdown the fetchers too early?

> ShuffleScheduler#allEventsReceived check is too tight
> -----------------------------------------------------
>
>                 Key: TEZ-3607
>                 URL: https://issues.apache.org/jira/browse/TEZ-3607
>             Project: Apache Tez
>          Issue Type: Sub-task
>            Reporter: Kuhu Shukla
>            Assignee: Kuhu Shukla
>         Attachments: TEZ-3607.001.patch
>
>
> In case of skipped records being non zero, the allEventsReceived() call can 
> return false even when all the events have in fact been received since 
> pathtoAttemptsMap size plus the skipped counter may be more than the total 
> number of inputs since we don't take out entries from the map if skipped 
> counter is incremented.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to