You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2021/12/01 20:04:16 UTC

[GitHub] [arrow] lidavidm commented on pull request #10260: ARROW-12671: [C++] Add OpenTelemetry to ThirdpartyToolchain

lidavidm commented on pull request #10260:
URL: https://github.com/apache/arrow/pull/10260#issuecomment-984010527


   > I'm curious: why are gRPC and protobuf required?
   
   This PR configures the OTLP exporter so we can dump collected data somewhere where we can analyze it. The OTLP exporter talks to the OTLP collector over either HTTP or gRPC, using Protobuf payloads, so at least Protobuf and one of cURL and gRPC is required. We only enable the HTTP protocol, but the upstream CMake config currently doesn't differentiate between whether you have gRPC enabled or not when generating Protobuf stubs, so gRPC is unconditionally required. (See https://github.com/open-telemetry/opentelemetry-cpp/issues/1045.) I think this may have just been fixed.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org