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

Siddharth Seth commented on HIVE-12915:
---------------------------------------

+1.
Couple of minor nits which you may want to fix in the commit patch.
- String[] defaultQueueList = defaultQueues.split(","); | Configuration has a 
method to get this trimmed - conf.getTrimmedStrings(). Would be better to use 
that, or whatever it uses.
- LOG.info("QueueName: " + queueName + " nonDefaultUser: " + nonDefaultUser +
          " defaultQueuePool: " + defaultQueuePool); | Also log 
hasInitialSessions

> Tez session pool has concurrency issues during init
> ---------------------------------------------------
>
>                 Key: HIVE-12915
>                 URL: https://issues.apache.org/jira/browse/HIVE-12915
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Takahiko Saito
>            Assignee: Sergey Shelukhin
>         Attachments: HIVE-12915.patch
>
>




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

Reply via email to