Jason Fehr has posted comments on this change. ( http://gerrit.cloudera.org:8080/21514 )
Change subject: IMPALA-13235: Add OpenTelemetry C++ ...................................................................... Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/21514/6/source/opentelemetry-cpp/build.sh File source/opentelemetry-cpp/build.sh: http://gerrit.cloudera.org:8080/#/c/21514/6/source/opentelemetry-cpp/build.sh@30 PS6, Line 30: -DCMAKE_BUILD_TYPE=RELEASE -DCMAKE_INSTALL_PREFIX=$LOCAL_INSTALL This cmake command needs a few other options set: -DCMAKE_EXPORT_COMPILE_COMMANDS=ON -DWITH_BENCHMARK=OFF -DWITH_EXAMPLES=OFF -DWITH_OTLP_GRPC=ON -DWITH_OTLP_HTTP=ON -DWITH_OTLP_HTTP_COMPRESSION=ON -- To view, visit http://gerrit.cloudera.org:8080/21514 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: native-toolchain Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I1657e2d344e90654b5d009d65973d762c0081a05 Gerrit-Change-Number: 21514 Gerrit-PatchSet: 6 Gerrit-Owner: Michael Smith <michael.sm...@cloudera.com> Gerrit-Reviewer: Jason Fehr <jf...@cloudera.com> Gerrit-Reviewer: Laszlo Gaal <laszlo.g...@cloudera.com> Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com> Gerrit-Reviewer: gaurav singh <gsi...@cloudera.com> Gerrit-Comment-Date: Wed, 14 May 2025 22:35:28 +0000 Gerrit-HasComments: Yes