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

Gopal V commented on HIVE-14608:
--------------------------------

[~sershe]: I think the original loop was written with the assumption that 
.canAcceptTask() can flip to true at some point during the loop.

That is probably a bad assumption now that we are running things in 
milliseconds - LGTM +1.

> LLAP: slow scheduling due to LlapTaskScheduler not removing nodes on kill 
> --------------------------------------------------------------------------
>
>                 Key: HIVE-14608
>                 URL: https://issues.apache.org/jira/browse/HIVE-14608
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Sergey Shelukhin
>            Assignee: Sergey Shelukhin
>            Priority: Critical
>         Attachments: HIVE-14608.patch
>
>
> See comments; this can result in a slowdown esp. if some critical task gets 
> unlucky.
> {noformat}
>   public void workerNodeRemoved(ServiceInstance serviceInstance) {
>      // FIXME: disabling this for now
> // instanceToNodeMap.remove(serviceInstance.getWorkerIdentity());
> {noformat}



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

Reply via email to