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

Change subject: IMPALA-14787: Add support for Redhat 10 and SLES 16
......................................................................


Patch Set 9:

(4 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/24459/9/bin/bootstrap_system.sh
File bin/bootstrap_system.sh:

http://gerrit.cloudera.org:8080/#/c/24459/9/bin/bootstrap_system.sh@261
PS9, Line 261:   sudo alternatives --set java_sdk_openjdk 
/usr/lib/jvm/java-${REDHAT_JAVA_VERSION}-openjdk
line too long (91 > 90)


http://gerrit.cloudera.org:8080/#/c/24459/9/bin/bootstrap_system.sh@268
PS9, Line 268:   sudo alternatives --set java_sdk_openjdk 
java-${REDHAT_JAVA_VERSION}-openjdk.${ARCH_NAME}
line too long (91 > 90)


http://gerrit.cloudera.org:8080/#/c/24459/9/bin/bootstrap_system.sh@297
PS9, Line 297:     sudo yum install -y 
https://dl.fedoraproject.org/pub/epel/epel-release-latest-10.noarch.rpm
line too long (95 > 90)


http://gerrit.cloudera.org:8080/#/c/24459/9/tests/custom_cluster/test_otel_trace.py
File tests/custom_cluster/test_otel_trace.py:

http://gerrit.cloudera.org:8080/#/c/24459/9/tests/custom_cluster/test_otel_trace.py@471
PS9, Line 471:
flake8: E501 line too long (92 > 90 characters)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I50acc5fbb4968e4310c4f84dde1d5b931b0385ec
Gerrit-Change-Number: 24459
Gerrit-PatchSet: 9
Gerrit-Owner: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Tue, 16 Jun 2026 06:13:28 +0000
Gerrit-HasComments: Yes

Reply via email to