[
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)