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

Rajesh Balamohan commented on TEZ-3317:
---------------------------------------

Sorry about the delay. Minor comments
- In OrderedGroupedKVInput, can {{shuffledInputs}} be init-ed in initialize() 
method itself like other counters?
- Should MapProcessor/ReduceProcessor update the progress only inputs.size > 
0?. 




> Speculative execution starts too early due to 0 progress
> --------------------------------------------------------
>
>                 Key: TEZ-3317
>                 URL: https://issues.apache.org/jira/browse/TEZ-3317
>             Project: Apache Tez
>          Issue Type: Improvement
>            Reporter: Jonathan Eagles
>            Assignee: Kuhu Shukla
>         Attachments: TEZ-3317.001.patch, TEZ-3317.002.patch, 
> TEZ-3317.003.patch, TEZ-3317.004.patch, TEZ-3317.005.patch, 
> TEZ-3317.006.patch, TEZ-3317.007.patch, TEZ-3317.008.patch, TEZ-3317.009.patch
>
>
> Don't know at this point if this is a tez or a PigProcessor issue. There is 
> some setProgress chain that is keeping task progress from being correctly 
> reported. Task status is always zero, so as soon as the first task finishes, 
> tasks up to the speculation limit are always launched.



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

Reply via email to