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

Reply via email to