Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/22527 )
Change subject: WIP: add ipv6 support for webui/hs2/hs2-http/beeswax ...................................................................... Patch Set 10: (45 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/22527/10/be/src/util/network-util.cc File be/src/util/network-util.cc: http://gerrit.cloudera.org:8080/#/c/22527/10/be/src/util/network-util.cc@101 PS10, Line 101: ss << "Could not convert IPv" << (is_ipv6_addr ? 6: 4) << "address for: " << hostname; line too long (92 > 90) http://gerrit.cloudera.org:8080/#/c/22527/10/tests/common/impala_cluster.py File tests/common/impala_cluster.py: http://gerrit.cloudera.org:8080/#/c/22527/10/tests/common/impala_cluster.py@561 PS10, Line 561: r flake8: E111 indentation is not a multiple of 2 http://gerrit.cloudera.org:8080/#/c/22527/10/tests/common/impala_service.py File tests/common/impala_service.py: http://gerrit.cloudera.org:8080/#/c/22527/10/tests/common/impala_service.py@80 PS10, Line 80: , flake8: E231 missing whitespace after ',' http://gerrit.cloudera.org:8080/#/c/22527/10/tests/common/impala_service.py@483 PS10, Line 483: c flake8: E501 line too long (100 > 90 characters) http://gerrit.cloudera.org:8080/#/c/22527/10/tests/common/impala_test_suite.py File tests/common/impala_test_suite.py: http://gerrit.cloudera.org:8080/#/c/22527/10/tests/common/impala_test_suite.py@373 PS10, Line 373: flake8: E261 at least two spaces before inline comment http://gerrit.cloudera.org:8080/#/c/22527/10/tests/common/impala_test_suite.py@373 PS10, Line 373: _ flake8: E501 line too long (100 > 90 characters) http://gerrit.cloudera.org:8080/#/c/22527/10/tests/common/impala_test_suite.py@495 PS10, Line 495: = flake8: E225 missing whitespace around operator http://gerrit.cloudera.org:8080/#/c/22527/10/tests/common/impala_test_suite.py@1782 PS10, Line 1782: c flake8: U100 Unused argument 'cls' http://gerrit.cloudera.org:8080/#/c/22527/10/tests/common/network.py File tests/common/network.py: http://gerrit.cloudera.org:8080/#/c/22527/10/tests/common/network.py@36 PS10, Line 36: def split_host_port(host_port): flake8: E302 expected 2 blank lines, found 1 http://gerrit.cloudera.org:8080/#/c/22527/10/tests/common/network.py@48 PS10, Line 48: flake8: W293 blank line contains whitespace http://gerrit.cloudera.org:8080/#/c/22527/10/tests/common/network.py@48 PS10, Line 48: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22527/10/tests/common/network.py@49 PS10, Line 49: def to_host_port(host, port): flake8: E302 expected 2 blank lines, found 1 http://gerrit.cloudera.org:8080/#/c/22527/10/tests/common/network.py@54 PS10, Line 54: CERT_TO_CA_MAP = { flake8: E305 expected 2 blank lines after class or function definition, found 1 http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py File tests/custom_cluster/test_ipv6.py: http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@24 PS10, Line 24: import re flake8: F401 're' imported but unused http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@26 PS10, Line 26: import signal flake8: F401 'signal' imported but unused http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@27 PS10, Line 27: import ssl flake8: F401 'ssl' imported but unused http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@28 PS10, Line 28: import socket flake8: F401 'socket' imported but unused http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@29 PS10, Line 29: import sys flake8: F401 'sys' imported but unused http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@30 PS10, Line 30: import time flake8: F401 'time' imported but unused http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@32 PS10, Line 32: from tests.common.environ import IS_REDHAT_DERIVATIVE flake8: F401 'tests.common.environ.IS_REDHAT_DERIVATIVE' imported but unused http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@34 PS10, Line 34: from tests.common.impala_service import ImpaladService flake8: F401 'tests.common.impala_service.ImpaladService' imported but unused http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@37 PS10, Line 37: from tests.shell.util import run_impala_shell_cmd, run_impala_shell_cmd_no_expect, \ flake8: F401 'tests.shell.util.run_impala_shell_cmd_no_expect' imported but unused http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@37 PS10, Line 37: from tests.shell.util import run_impala_shell_cmd, run_impala_shell_cmd_no_expect, \ flake8: F401 'tests.shell.util.ImpalaShell' imported but unused http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@62 PS10, Line 62: R flake8: E501 line too long (93 > 90 characters) http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@63 PS10, Line 63: R flake8: E501 line too long (93 > 90 characters) http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@74 PS10, Line 74: flake8: E241 multiple spaces after ':' http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@81 PS10, Line 81: flake8: E241 multiple spaces after ':' http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@88 PS10, Line 88: class TestIPv6Base(CustomClusterTestSuite): flake8: E302 expected 2 blank lines, found 1 http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@104 PS10, Line 104: i flake8: E225 missing whitespace around operator http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@133 PS10, Line 133: i flake8: E225 missing whitespace around operator http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@146 PS10, Line 146: e flake8: E501 line too long (96 > 90 characters) http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@153 PS10, Line 153: U flake8: E501 line too long (99 > 90 characters) http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@179 PS10, Line 179: U flake8: E501 line too long (99 > 90 characters) http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@206 PS10, Line 206: L flake8: E501 line too long (135 > 90 characters) http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@207 PS10, Line 207: W flake8: E501 line too long (108 > 90 characters) http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@208 PS10, Line 208: D flake8: E501 line too long (105 > 90 characters) http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@235 PS10, Line 235: Y flake8: E501 line too long (135 > 90 characters) http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@236 PS10, Line 236: flake8: E251 unexpected spaces around keyword / parameter equals http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@236 PS10, Line 236: W flake8: E501 line too long (108 > 90 characters) http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@237 PS10, Line 237: flake8: E251 unexpected spaces around keyword / parameter equals http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@237 PS10, Line 237: D flake8: E501 line too long (105 > 90 characters) http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_ipv6.py@247 PS10, Line 247: flake8: E241 multiple spaces after ',' http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_redaction.py File tests/custom_cluster/test_redaction.py: http://gerrit.cloudera.org:8080/#/c/22527/10/tests/custom_cluster/test_redaction.py@27 PS10, Line 27: from tests.common.impala_test_suite import ImpalaTestSuite flake8: F401 'tests.common.impala_test_suite.ImpalaTestSuite' imported but unused http://gerrit.cloudera.org:8080/#/c/22527/10/tests/stress/test_insert_stress.py File tests/stress/test_insert_stress.py: http://gerrit.cloudera.org:8080/#/c/22527/10/tests/stress/test_insert_stress.py@38 PS10, Line 38: flake8: W293 blank line contains whitespace http://gerrit.cloudera.org:8080/#/c/22527/10/tests/stress/test_insert_stress.py@38 PS10, Line 38: line has trailing whitespace -- To view, visit http://gerrit.cloudera.org:8080/22527 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I51ac66c568cc9bb06f4a3915db07a53c100109b6 Gerrit-Change-Number: 22527 Gerrit-PatchSet: 10 Gerrit-Owner: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Abhishek Rawat <ara...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Kurt Deschler <kdesc...@cloudera.com> Gerrit-Reviewer: gaurav singh <gsi...@cloudera.com> Gerrit-Comment-Date: Tue, 11 Mar 2025 14:58:31 +0000 Gerrit-HasComments: Yes
