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

Kuhu Shukla commented on TEZ-3944:
----------------------------------

Thank you for the patch [~jeagles]. It looks good with just one minor nit in 
testTaskSchedulerPreemption :

{code}
ContainerId mockCId5 = ContainerId.newContainerId(attemptId, 5);
    Container mockContainer5 = Container.newInstance(mockCId5, host1, null, 
taskAsk, pri8, null);
    containers.clear();
    containers.add(mockContainer5);
{code}
These two variables should be named {{mockCId4}} and {{mockContainer4}} 
respectively.

> TestTaskScheduler times-out on Hadoop3
> --------------------------------------
>
>                 Key: TEZ-3944
>                 URL: https://issues.apache.org/jira/browse/TEZ-3944
>             Project: Apache Tez
>          Issue Type: Sub-task
>            Reporter: Eric Wohlstadter
>            Assignee: Jonathan Eagles
>            Priority: Major
>         Attachments: TEZ-3944.001.patch, TEZ-3944.002.patch, 
> org.apache.tez.dag.app.rm.TestTaskScheduler-output.txt
>
>
> TestTaskScheduler times-out intermittently.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to