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

Change subject: IMPALA-13835: Remove reference to protocol-specific states
......................................................................


Patch Set 1:

(3 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/22586/1/tests/custom_cluster/test_events_custom_configs.py
File tests/custom_cluster/test_events_custom_configs.py:

http://gerrit.cloudera.org:8080/#/c/22586/1/tests/custom_cluster/test_events_custom_configs.py@668
PS1, Line 668: ,
flake8: E231 missing whitespace after ','


http://gerrit.cloudera.org:8080/#/c/22586/1/tests/metadata/test_load.py
File tests/metadata/test_load.py:

http://gerrit.cloudera.org:8080/#/c/22586/1/tests/metadata/test_load.py@236
PS1, Line 236: (
flake8: E275 missing whitespace after keyword


http://gerrit.cloudera.org:8080/#/c/22586/1/tests/metadata/test_load.py@245
PS1, Line 245: (
flake8: E275 missing whitespace after keyword



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Id6b56024fbfcea1ff005c34cd146d16e67cb6fa1
Gerrit-Change-Number: 22586
Gerrit-PatchSet: 1
Gerrit-Owner: Riza Suminto <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Thu, 06 Mar 2025 01:40:54 +0000
Gerrit-HasComments: Yes

Reply via email to