You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by se...@apache.org on 2016/09/09 02:07:57 UTC

hive git commit: HIVE-14608 : LLAP: slow scheduling due to LlapTaskScheduler not removing nodes on kill (Sergey Shelukhin, reviewed by Gopal V) ADDENDUM

Repository: hive
Updated Branches:
  refs/heads/branch-2.1 06fd3a95a -> 50505126e


HIVE-14608 : LLAP: slow scheduling due to LlapTaskScheduler not removing nodes on kill (Sergey Shelukhin, reviewed by Gopal V) ADDENDUM


Project: http://git-wip-us.apache.org/repos/asf/hive/repo
Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/50505126
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/50505126
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/50505126

Branch: refs/heads/branch-2.1
Commit: 50505126effa05101db62249fbf13f44ef62b0a5
Parents: 06fd3a9
Author: Sergey Shelukhin <se...@apache.org>
Authored: Thu Sep 8 18:56:39 2016 -0700
Committer: Sergey Shelukhin <se...@apache.org>
Committed: Thu Sep 8 18:56:39 2016 -0700

----------------------------------------------------------------------
 .../hadoop/hive/llap/tezplugins/LlapTaskSchedulerService.java      | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/50505126/llap-tez/src/java/org/apache/hadoop/hive/llap/tezplugins/LlapTaskSchedulerService.java
----------------------------------------------------------------------
diff --git a/llap-tez/src/java/org/apache/hadoop/hive/llap/tezplugins/LlapTaskSchedulerService.java b/llap-tez/src/java/org/apache/hadoop/hive/llap/tezplugins/LlapTaskSchedulerService.java
index 5f739fe..3b319e9 100644
--- a/llap-tez/src/java/org/apache/hadoop/hive/llap/tezplugins/LlapTaskSchedulerService.java
+++ b/llap-tez/src/java/org/apache/hadoop/hive/llap/tezplugins/LlapTaskSchedulerService.java
@@ -783,7 +783,7 @@ public class LlapTaskSchedulerService extends TaskScheduler {
         }
       }
       /* fall through - miss in locality (random scheduling) or no locality-requested */
-      Collection<ServiceInstance> instances = activeInstances.getAll();
+      Collection<ServiceInstance> instances = activeInstances.getAll().values();
       ArrayList<NodeInfo> all = new ArrayList<>(instances.size());
       for (ServiceInstance inst : instances) {
         NodeInfo nodeInfo = instanceToNodeMap.get(inst.getWorkerIdentity());