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

Gopal V commented on HIVE-11969:
--------------------------------

[~sershe]: LGTM - +1.

Minor comment on the isOpenOrOpening() - can you split that up into 2 different 
functions, so that the isOpen() does not change.

That allows the if(isOpening()) to cancel the future & the isOpen() to close 
the session.

> start Tez session in background when starting CLI
> -------------------------------------------------
>
>                 Key: HIVE-11969
>                 URL: https://issues.apache.org/jira/browse/HIVE-11969
>             Project: Hive
>          Issue Type: Bug
>          Components: Tez
>            Reporter: Sergey Shelukhin
>            Assignee: Sergey Shelukhin
>         Attachments: HIVE-11969.01.patch, HIVE-11969.02.patch, 
> HIVE-11969.03.patch, HIVE-11969.patch, Screen Shot 2015-10-02 at 14.23.17 .png
>
>
> Tez session spins up AM, which can cause delays, esp. if the cluster is very 
> busy.
> This can be done in background, so the AM might get started while the user is 
> running local commands and doing other things.



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

Reply via email to