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

Change subject: IMPALA-5924: Formatting TBLPROPERTIES in SHOW CREATE TABLE
......................................................................


Patch Set 9:

(7 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/24554/9/tests/custom_cluster/test_workload_mgmt_init.py
File tests/custom_cluster/test_workload_mgmt_init.py:

http://gerrit.cloudera.org:8080/#/c/24554/9/tests/custom_cluster/test_workload_mgmt_init.py@112
PS9, Line 112:
flake8: W293 blank line contains whitespace


http://gerrit.cloudera.org:8080/#/c/24554/9/tests/custom_cluster/test_workload_mgmt_init.py@112
PS9, Line 112:
line has trailing whitespace


http://gerrit.cloudera.org:8080/#/c/24554/9/tests/custom_cluster/test_workload_mgmt_init.py@118
PS9, Line 118: )
flake8: E501 line too long (91 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/24554/9/tests/custom_cluster/test_workload_mgmt_init.py@120
PS9, Line 120:
flake8: W293 blank line contains whitespace


http://gerrit.cloudera.org:8080/#/c/24554/9/tests/custom_cluster/test_workload_mgmt_init.py@120
PS9, Line 120:
line has trailing whitespace


http://gerrit.cloudera.org:8080/#/c/24554/9/tests/custom_cluster/test_workload_mgmt_init.py@123
PS9, Line 123:   
flake8: W293 blank line contains whitespace


http://gerrit.cloudera.org:8080/#/c/24554/9/tests/custom_cluster/test_workload_mgmt_init.py@123
PS9, Line 123:
line has trailing whitespace



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic198d0a42cf3636fe612460c12d8c8000a313ef1
Gerrit-Change-Number: 24554
Gerrit-PatchSet: 9
Gerrit-Owner: Gowthami Bisati <[email protected]>
Gerrit-Reviewer: Abhishek Talluri <[email protected]>
Gerrit-Reviewer: Gowthami Bisati <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
Gerrit-Comment-Date: Thu, 23 Jul 2026 08:32:28 +0000
Gerrit-HasComments: Yes

Reply via email to