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

Siddharth Seth commented on HIVE-17256:
---------------------------------------

Mostly looks good. Would be nice to have a few more tests on the ordering of 
the various  queues / or even better the scheduler making correct decisions.

A guaranteed task will also replace a non-guaranteed task, irrespective of 
finishable state? Wasn't there some potential for deadlocks with this?

In terms of the todo wtf - iirc fixing that requires making some biggish 
changes in tez internals to prevent the same finish being registered multiple 
times over. Would be good to leave that as a comment if it does not exist, 
instead of the wtf. The original jira where that was added should have more 
context.


> add a notion of a guaranteed task to LLAP
> -----------------------------------------
>
>                 Key: HIVE-17256
>                 URL: https://issues.apache.org/jira/browse/HIVE-17256
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Sergey Shelukhin
>            Assignee: Sergey Shelukhin
>         Attachments: HIVE-17256.patch
>
>
> Tasks are basically on two levels, guaranteed and speculative, with 
> speculative being the default. As long as noone uses the new flag, the tasks 
> behave the same.
> All the tasks that do have the flag also behave the same with regard to each 
> other.
> The difference is that a guaranteed task is always higher priority, and 
> preempts, a speculative task. 



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

Reply via email to