Michael Smith has posted comments on this change. ( http://gerrit.cloudera.org:8080/22924 )
Change subject: IMPALA-13235: Update the build toolchain to include the latest OpenTelemetry C++ sdk. ...................................................................... Patch Set 2: (1 comment) > Patch Set 2: > > Build Failed > > https://jenkins.impala.io/job/gerrit-code-review-checks/19279/ : Initial code > review checks failed. See linked job for details on the failure. You'll need to rebase to pick up IMPALA-14067. http://gerrit.cloudera.org:8080/#/c/22924/2/bin/impala-config.sh File bin/impala-config.sh: http://gerrit.cloudera.org:8080/#/c/22924/2/bin/impala-config.sh@90 PS2, Line 90: ${IMPALA_TOOLCHAIN_COMMIT_HASH-49a326be6f1836127b1189cea71ea9832b31b7aa} > We should also update this to match, f9f56a8a24 (or the full hash). We probably could infer this from ${IMPALA_TOOLCHAIN_BUILD_ID_X86_64#*-}, which we test below. -- To view, visit http://gerrit.cloudera.org:8080/22924 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie40b5cd33274df13f3005bf7a704299ebfff8a5b Gerrit-Change-Number: 22924 Gerrit-PatchSet: 2 Gerrit-Owner: gaurav singh <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Comment-Date: Thu, 22 May 2025 18:43:46 +0000 Gerrit-HasComments: Yes
