Vihang Karajgaonkar has posted comments on this change. ( http://gerrit.cloudera.org:8080/13933 )
Change subject: IMPALA-8627: Enable catalog-v2 in tests ...................................................................... Patch Set 14: (4 comments) http://gerrit.cloudera.org:8080/#/c/13933/13/tests/hs2/hs2_test_suite.py File tests/hs2/hs2_test_suite.py: http://gerrit.cloudera.org:8080/#/c/13933/13/tests/hs2/hs2_test_suite.py@29 PS13, Line 29: > flake8: E302 expected 2 blank lines, found 1 Done http://gerrit.cloudera.org:8080/#/c/13933/13/tests/hs2/hs2_test_suite.py@72 PS13, Line 72: # decorators without installing new modules. > flake8: E302 expected 2 blank lines, found 1 Done http://gerrit.cloudera.org:8080/#/c/13933/13/tests/metadata/test_metadata_query_statements.py File tests/metadata/test_metadata_query_statements.py: http://gerrit.cloudera.org:8080/#/c/13933/13/tests/metadata/test_metadata_query_statements.py@180 PS13, Line 180: > flake8: E501 line too long (92 > 90 characters) Done http://gerrit.cloudera.org:8080/#/c/13933/13/tests/metadata/test_metadata_query_statements.py@181 PS13, Line 181: > flake8: E501 line too long (91 > 90 characters) Done -- 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: 14 Gerrit-Owner: Vihang Karajgaonkar <vih...@cloudera.com> Gerrit-Reviewer: Bharath Vissapragada <bhara...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Vihang Karajgaonkar <vih...@cloudera.com> Gerrit-Comment-Date: Tue, 06 Aug 2019 19:38:50 +0000 Gerrit-HasComments: Yes