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

Rajesh Balamohan commented on TEZ-1742:
---------------------------------------

Any reason why request is added to highestPriRequests when the priorities are 
the same
{code}
else if (request.getPriority().equals(highestPriRequest.getPriority())) {
          highestPriRequests.add(request);
        }
{code}


{code}
List<CookieContainerRequest> highestPriRequests
{code}
This is used for checking the number of highest priority requests. Is it 
possible to maintain an int instead of list?

Please remove the unused LinkedHashSet import statement. 

> Improve response time of internal preemption
> --------------------------------------------
>
>                 Key: TEZ-1742
>                 URL: https://issues.apache.org/jira/browse/TEZ-1742
>             Project: Apache Tez
>          Issue Type: Task
>            Reporter: Bikas Saha
>            Assignee: Bikas Saha
>         Attachments: TEZ-1742.1.patch
>
>
> Tez YARN Task Scheduler currently preempts 1 running task at a time when a 
> higher priority task is waiting and there are no available resources. When a 
> large number of higher priority tasks are pending then it can take a long 
> time to preempt the required number of lower priority tasks.



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

Reply via email to