Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/14059 )
Change subject: IMPALA-8863: Add support to run tests over HTTP/HS2 ...................................................................... Patch Set 25: Code-Review+2 (1 comment) Picking this back up. The change now passes our precommit checks and also passed a private parameterized build. I undid the debug statements that I had put in place since PS16 (Tim's +2). Carrying his +2 here. http://gerrit.cloudera.org:8080/#/c/14059/24/tests/common/impala_test_suite.py File tests/common/impala_test_suite.py: http://gerrit.cloudera.org:8080/#/c/14059/24/tests/common/impala_test_suite.py@272 PS24, Line 272: > flake8: E303 too many blank lines (2) Done -- To view, visit http://gerrit.cloudera.org:8080/14059 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I7156558071781378fcb9c8941c0f4dd82eb0d018 Gerrit-Change-Number: 14059 Gerrit-PatchSet: 25 Gerrit-Owner: Lars Volker <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Lars Volker <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Comment-Date: Sat, 16 Nov 2019 01:10:46 +0000 Gerrit-HasComments: Yes
