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 12: (5 comments) http://gerrit.cloudera.org:8080/#/c/13933/12/tests/hs2/hs2_test_suite.py File tests/hs2/hs2_test_suite.py: http://gerrit.cloudera.org:8080/#/c/13933/12/tests/hs2/hs2_test_suite.py@29 PS12, Line 29: def add_session_helper(self, protocol_version, conf_overlay, close_session, fn): flake8: E302 expected 2 blank lines, found 1 http://gerrit.cloudera.org:8080/#/c/13933/12/tests/hs2/hs2_test_suite.py@72 PS12, Line 72: def needs_session_cluster_properties(protocol_version= flake8: E302 expected 2 blank lines, found 1 http://gerrit.cloudera.org:8080/#/c/13933/12/tests/hs2/hs2_test_suite.py@72 PS12, Line 72: flake8: E251 unexpected spaces around keyword / parameter equals http://gerrit.cloudera.org:8080/#/c/13933/12/tests/metadata/test_metadata_query_statements.py File tests/metadata/test_metadata_query_statements.py: http://gerrit.cloudera.org:8080/#/c/13933/12/tests/metadata/test_metadata_query_statements.py@180 PS12, Line 180: 0 flake8: E501 line too long (92 > 90 characters) http://gerrit.cloudera.org:8080/#/c/13933/12/tests/metadata/test_metadata_query_statements.py@181 PS12, 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: 12 Gerrit-Owner: Vihang Karajgaonkar <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Reviewer: Vihang Karajgaonkar <[email protected]> Gerrit-Comment-Date: Tue, 06 Aug 2019 19:08:28 +0000 Gerrit-HasComments: Yes
