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

Siddharth Seth commented on HIVE-15958:
---------------------------------------

The stopUmbilical in hasAmFaield() - is this required? Think this has the 
potential to cause an NPE if there's a taskKilled already queued up.

In QueryTracker - instead of introducing a new map to store the NodeId - the 
NodeId can be stored in QueryInfo.

Otherwise, looks good.

> LLAP: IPC connections are not being reused for umbilical protocol
> -----------------------------------------------------------------
>
>                 Key: HIVE-15958
>                 URL: https://issues.apache.org/jira/browse/HIVE-15958
>             Project: Hive
>          Issue Type: Bug
>          Components: llap
>    Affects Versions: 2.2.0
>            Reporter: Rajesh Balamohan
>            Assignee: Prasanth Jayachandran
>         Attachments: HIVE-15958.1.patch, HIVE-15958.2.patch, 
> HIVE-15958.3.patch, HIVE-15958.4.patch, HIVE-15958.4.patch
>
>
> During concurrency testing, observed 1000s of ipc thread creations. Ideally, 
> the connections to same hosts should be reused.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to