Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/18125 )

Change subject: IMPALA-11070: Create TestSection class.
......................................................................


Patch Set 2:

(4 comments)

http://gerrit.cloudera.org:8080/#/c/18125/2/tests/common/impala_test_suite.py
File tests/common/impala_test_suite.py:

http://gerrit.cloudera.org:8080/#/c/18125/2/tests/common/impala_test_suite.py@693
PS2, Line 693: u
flake8: F841 local variable 'user' is assigned to but never used


http://gerrit.cloudera.org:8080/#/c/18125/2/tests/common/impala_test_suite.py@703
PS2, Line 703: "
flake8: E126 continuation line over-indented for hanging indent


http://gerrit.cloudera.org:8080/#/c/18125/2/tests/common/impala_test_suite.py@732
PS2, Line 732: \
flake8: E502 the backslash is redundant between brackets


http://gerrit.cloudera.org:8080/#/c/18125/2/tests/common/test_result_verifier.py
File tests/common/test_result_verifier.py:

http://gerrit.cloudera.org:8080/#/c/18125/2/tests/common/test_result_verifier.py@431
PS2, Line 431: \
flake8: E502 the backslash is redundant between brackets



--
To view, visit http://gerrit.cloudera.org:8080/18125
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3fed7ef64ac263b49b5b6e4a237a296092e4dcb8
Gerrit-Change-Number: 18125
Gerrit-PatchSet: 2
Gerrit-Owner: Steve Carlin <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Thu, 06 Jan 2022 20:26:08 +0000
Gerrit-HasComments: Yes

Reply via email to