Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/22358 )
Change subject: IMPALA-13672: Migrate query_test/test_kudu.py to use hs2 protocol ...................................................................... Patch Set 1: (6 comments) gerrit-auto-critic failed. You can reproduce it locally using command: python3 bin/jenkins/critique-gerrit-review.py --dryrun To run it, you might need a virtual env with Python3's venv installed. http://gerrit.cloudera.org:8080/#/c/22358/1/tests/custom_cluster/test_kudu.py File tests/custom_cluster/test_kudu.py: http://gerrit.cloudera.org:8080/#/c/22358/1/tests/custom_cluster/test_kudu.py@49 PS1, Line 49: . flake8: E501 line too long (91 > 90 characters) http://gerrit.cloudera.org:8080/#/c/22358/1/tests/query_test/test_kudu.py File tests/query_test/test_kudu.py: http://gerrit.cloudera.org:8080/#/c/22358/1/tests/query_test/test_kudu.py@603 PS1, Line 603: a flake8: E501 line too long (92 > 90 characters) http://gerrit.cloudera.org:8080/#/c/22358/1/tests/query_test/test_kudu.py@618 PS1, Line 618: a flake8: E501 line too long (92 > 90 characters) http://gerrit.cloudera.org:8080/#/c/22358/1/tests/query_test/test_kudu.py@1667 PS1, Line 1667: , flake8: E501 line too long (91 > 90 characters) http://gerrit.cloudera.org:8080/#/c/22358/1/tests/query_test/test_kudu.py@1675 PS1, Line 1675: 1 flake8: E501 line too long (93 > 90 characters) http://gerrit.cloudera.org:8080/#/c/22358/1/tests/query_test/test_kudu.py@1680 PS1, Line 1680: ) flake8: E501 line too long (91 > 90 characters) -- To view, visit http://gerrit.cloudera.org:8080/22358 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I5f38baf5a0bbde1a1ad0bb4666c300f4f3cabd33 Gerrit-Change-Number: 22358 Gerrit-PatchSet: 1 Gerrit-Owner: Riza Suminto <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Comment-Date: Thu, 16 Jan 2025 03:35:43 +0000 Gerrit-HasComments: Yes
