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

Change subject: IMPALA-12870: Tag query id for Java pool threads
......................................................................


Patch Set 6:

(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/23558/6/tests/custom_cluster/test_observability.py
File tests/custom_cluster/test_observability.py:

http://gerrit.cloudera.org:8080/#/c/23558/6/tests/custom_cluster/test_observability.py@57
PS6, Line 57: %
flake8: E131 continuation line unaligned for hanging indent


http://gerrit.cloudera.org:8080/#/c/23558/6/tests/custom_cluster/test_observability.py@62
PS6, Line 62: %
flake8: E131 continuation line unaligned for hanging indent


http://gerrit.cloudera.org:8080/#/c/23558/6/tests/custom_cluster/test_observability.py@73
PS6, Line 73: %
flake8: E131 continuation line unaligned for hanging indent


http://gerrit.cloudera.org:8080/#/c/23558/6/tests/custom_cluster/test_observability.py@76
PS6, Line 76: %
flake8: E131 continuation line unaligned for hanging indent


http://gerrit.cloudera.org:8080/#/c/23558/6/tests/custom_cluster/test_observability.py@79
PS6, Line 79: %
flake8: E131 continuation line unaligned for hanging indent


http://gerrit.cloudera.org:8080/#/c/23558/6/tests/custom_cluster/test_observability.py@82
PS6, Line 82: %
flake8: E131 continuation line unaligned for hanging indent



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I83cca55edc72de35f5e8c5422efc104e6aa894c1
Gerrit-Change-Number: 23558
Gerrit-PatchSet: 6
Gerrit-Owner: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Mon, 20 Oct 2025 13:24:31 +0000
Gerrit-HasComments: Yes

Reply via email to