Riza Suminto has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22577 )
Change subject: IMPALA-13822: Add more detail in impala_connection.py logs ...................................................................... IMPALA-13822: Add more detail in impala_connection.py logs This patch make impala_connection.py to use the same log format as declared in conftest.py. Connection specific logs will have the protocol name printed. Modified set_configuration() and set_configuration_option() to make option related logging more concise. Moved LOG_FORMAT from conftest.py to patterns.py for reuse in impala_connection.py. Testing: - Run TestExprLimits locally and confirm that the log lines printed at logs/ee_tests/results/TEST-impala-parallel.xml is OK. Change-Id: I44ea7fbec15684ac5379703f781a400b4f17da8d Reviewed-on: http://gerrit.cloudera.org:8080/22577 Tested-by: Impala Public Jenkins <[email protected]> Reviewed-by: Csaba Ringhofer <[email protected]> --- M tests/common/impala_connection.py M tests/common/impala_test_suite.py M tests/common/patterns.py M tests/conftest.py 4 files changed, 64 insertions(+), 39 deletions(-) Approvals: Impala Public Jenkins: Verified Csaba Ringhofer: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/22577 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I44ea7fbec15684ac5379703f781a400b4f17da8d Gerrit-Change-Number: 22577 Gerrit-PatchSet: 6 Gerrit-Owner: Riza Suminto <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Daniel Becker <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]>
