[impala] branch master updated: IMPALA-8747: Fix string formatting on HS2 connection setup failure

[jo...@apache.org]

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

joemcdonnell 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 0fedc1f  IMPALA-8747: Fix string formatting on HS2 connection setup failure
0fedc1f is described below

commit 0fedc1f97f9571e59b8b1dc3ed963d44d9ac3f03
Author: Joe McDonnell <jo...@cloudera.com>
AuthorDate: Mon Jul 8 15:12:54 2019 -0700

    IMPALA-8747: Fix string formatting on HS2 connection setup failure
    
    In impala_test_suite.py, the logging for HS2 connection failure
    uses "{0}" replacement codes when the logging facility expects it
    to use "%"-style codes. This fixes it to call .format() directly.
    
    Change-Id: I4789712fcd71c887a9c6c890cd22a4f382faa3a5
    Reviewed-on: http://gerrit.cloudera.org:8080/13815
    Reviewed-by: Impala Public Jenkins <im...@cloudera.com>
    Tested-by: Impala Public Jenkins <im...@cloudera.com>
---
 tests/common/impala_test_suite.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/common/impala_test_suite.py b/tests/common/impala_test_suite.py
index f1db981..9ba1251 100644
--- a/tests/common/impala_test_suite.py
+++ b/tests/common/impala_test_suite.py
@@ -164,7 +164,7 @@ class ImpalaTestSuite(BaseTestSuite):
       cls.hs2_client = cls.create_impala_client(protocol='hs2')
     except Exception, e:
       # HS2 connection can fail for benign reasons, e.g. running with unsupported auth.
-      LOG.info("HS2 connection setup failed, continuing...: {0}", e)
+      LOG.info("HS2 connection setup failed, continuing...: {0}".format(e))
 
     # Default query options are populated on demand.
     cls.default_query_options = {}