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