[impala] branch master updated: IMPALA-8932: addendum - protocol var not defined

[ta...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

tarmstrong pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git


The following commit(s) were added to refs/heads/master by this push:
     new e070dbb  IMPALA-8932: addendum - protocol var not defined
e070dbb is described below

commit e070dbb02c264e26df576b40b8b792d02619fc60
Author: Tim Armstrong <ta...@cloudera.com>
AuthorDate: Thu Sep 12 11:22:20 2019 -0700

    IMPALA-8932: addendum - protocol var not defined
    
    Change-Id: I75c41a02bc7f1314e48bb5a39b945119264ce478
    Reviewed-on: http://gerrit.cloudera.org:8080/14225
    Reviewed-by: Impala Public Jenkins <im...@cloudera.com>
    Tested-by: Impala Public Jenkins <im...@cloudera.com>
---
 shell/impala_shell.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/shell/impala_shell.py b/shell/impala_shell.py
index a52b8e0..35af848 100755
--- a/shell/impala_shell.py
+++ b/shell/impala_shell.py
@@ -790,12 +790,12 @@ class ImpalaShell(object, cmd.Cmd):
     tokens = args.split(" ")
     # validate the connection string.
     host_port = [val for val in tokens[0].split(':') if val.strip()]
+    protocol = options.protocol.lower()
     if (':' in tokens[0] and len(host_port) != 2):
       print_to_stderr("Connection string must either be empty, or of the form "
                       "<hostname[:port]>")
       return CmdStatus.ERROR
     elif len(host_port) == 1:
-      protocol = options.protocol.lower()
       if protocol == 'hs2':
         port = str(DEFAULT_HS2_PORT)
       elif protocol == 'hs2-http':