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

Siddharth Seth commented on TEZ-3880:
-------------------------------------

Is the test failure related? Otherwise, the patch looks good to me.
One thing that is likely not being handled is the case where the executors 
accept work, and then reject/preempt them before execution - that is more like 
a rejection than a preemption.

> do not count rejected tasks as killed in vertex progress
> --------------------------------------------------------
>
>                 Key: TEZ-3880
>                 URL: https://issues.apache.org/jira/browse/TEZ-3880
>             Project: Apache Tez
>          Issue Type: Task
>            Reporter: Sergey Shelukhin
>            Assignee: Sergey Shelukhin
>         Attachments: TEZ-3880.01.patch, TEZ-3880.patch
>
>
> Tasks rejected from LLAP because the cluster is full are shown as killed 
> tasks in the commandline query UI (CLI and beeline). This shouldn't really 
> happen; killed tasks in the container case means something else, and this 
> scenario doesn't exist because AM doesn't continuously try to queue tasks. We 
> could change LLAP queue to use sort of a pull model (would also allow for 
> better duplicate scheduling), but for now we should fix the UI



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to