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

Change subject: IMPALA-8627: Enable catalog-v2 in tests
......................................................................


Patch Set 10:

(4 comments)

http://gerrit.cloudera.org:8080/#/c/13933/10/tests/hs2/hs2_test_suite.py
File tests/hs2/hs2_test_suite.py:

http://gerrit.cloudera.org:8080/#/c/13933/10/tests/hs2/hs2_test_suite.py@66
PS10, Line 66: def needs_session_cluster_properties(protocol_version=
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/13933/10/tests/hs2/hs2_test_suite.py@66
PS10, Line 66:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/13933/10/tests/metadata/test_metadata_query_statements.py
File tests/metadata/test_metadata_query_statements.py:

http://gerrit.cloudera.org:8080/#/c/13933/10/tests/metadata/test_metadata_query_statements.py@180
PS10, Line 180: 0
flake8: E501 line too long (92 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/13933/10/tests/metadata/test_metadata_query_statements.py@181
PS10, Line 181: t
flake8: E501 line too long (91 > 90 characters)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iddbde666de2b780c0e40df716a9dfe54524e092d
Gerrit-Change-Number: 13933
Gerrit-PatchSet: 10
Gerrit-Owner: Vihang Karajgaonkar <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Vihang Karajgaonkar <[email protected]>
Gerrit-Comment-Date: Sun, 04 Aug 2019 23:57:52 +0000
Gerrit-HasComments: Yes

Reply via email to