Michael Smith has posted comments on this change. ( http://gerrit.cloudera.org:8080/21514 )
Change subject: IMPALA-13235: Add OpenTelemetry C++ ...................................................................... Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/21514/7/source/opentelemetry-cpp/build.sh File source/opentelemetry-cpp/build.sh: http://gerrit.cloudera.org:8080/#/c/21514/7/source/opentelemetry-cpp/build.sh@36 PS7, Line 36: -DWITH_OTLP_GRPC=ON \ Do we need gRPC support? We don't use it. And we don't have a build of gRPC as part of the native toolchain. -- 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: 7 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: Thu, 15 May 2025 17:50:44 +0000 Gerrit-HasComments: Yes
