Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/19552 )
Change subject: IMPALA-11952 (part 2): Fix print function syntax ...................................................................... Patch Set 2: (13 comments) http://gerrit.cloudera.org:8080/#/c/19552/2/bin/compare_branches.py File bin/compare_branches.py: http://gerrit.cloudera.org:8080/#/c/19552/2/bin/compare_branches.py@273 PS2, Line 273: \ > flake8: E502 the backslash is redundant between brackets Done http://gerrit.cloudera.org:8080/#/c/19552/2/bin/push_to_asf.py File bin/push_to_asf.py: http://gerrit.cloudera.org:8080/#/c/19552/2/bin/push_to_asf.py@119 PS2, Line 119: \ > flake8: E502 the backslash is redundant between brackets Done http://gerrit.cloudera.org:8080/#/c/19552/2/bin/push_to_asf.py@139 PS2, Line 139: \ > flake8: E502 the backslash is redundant between brackets Done http://gerrit.cloudera.org:8080/#/c/19552/2/bin/push_to_asf.py@147 PS2, Line 147: G > flake8: F821 undefined name 'GERRIT_HOST' Fixed this to refer to GERRIT_URL_RE http://gerrit.cloudera.org:8080/#/c/19552/2/bin/push_to_asf.py@218 PS2, Line 218: d > flake8: E501 line too long (95 > 90 characters) Done http://gerrit.cloudera.org:8080/#/c/19552/2/bin/push_to_asf.py@227 PS2, Line 227: \ > flake8: E502 the backslash is redundant between brackets Done http://gerrit.cloudera.org:8080/#/c/19552/2/bin/push_to_asf.py@235 PS2, Line 235: \ > flake8: E502 the backslash is redundant between brackets Done http://gerrit.cloudera.org:8080/#/c/19552/2/bin/push_to_asf.py@257 PS2, Line 257: \ > flake8: E502 the backslash is redundant between brackets Done http://gerrit.cloudera.org:8080/#/c/19552/2/infra/deploy/deploy.py File infra/deploy/deploy.py: http://gerrit.cloudera.org:8080/#/c/19552/2/infra/deploy/deploy.py@452 PS2, Line 452: ) > flake8: E501 line too long (91 > 90 characters) Done http://gerrit.cloudera.org:8080/#/c/19552/2/testdata/bin/generate-schema-statements.py File testdata/bin/generate-schema-statements.py: http://gerrit.cloudera.org:8080/#/c/19552/2/testdata/bin/generate-schema-statements.py@642 PS2, Line 642: a > flake8: E501 line too long (95 > 90 characters) Done http://gerrit.cloudera.org:8080/#/c/19552/2/testdata/bin/generate-schema-statements.py@839 PS2, Line 839: \ > flake8: E502 the backslash is redundant between brackets Done http://gerrit.cloudera.org:8080/#/c/19552/2/testdata/bin/generate-test-vectors.py File testdata/bin/generate-test-vectors.py: http://gerrit.cloudera.org:8080/#/c/19552/2/testdata/bin/generate-test-vectors.py@119 PS2, Line 119: \ > flake8: E502 the backslash is redundant between brackets Done http://gerrit.cloudera.org:8080/#/c/19552/2/tests/query_test/test_aggregation.py File tests/query_test/test_aggregation.py: http://gerrit.cloudera.org:8080/#/c/19552/2/tests/query_test/test_aggregation.py@154 PS2, Line 154: , > flake8: E231 missing whitespace after ',' Done -- To view, visit http://gerrit.cloudera.org:8080/19552 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib634958369ad777a41e72d80c8053b74384ac351 Gerrit-Change-Number: 19552 Gerrit-PatchSet: 2 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Comment-Date: Mon, 27 Feb 2023 19:44:50 +0000 Gerrit-HasComments: Yes
