cpcloud commented on a change in pull request #11925:
URL: https://github.com/apache/arrow/pull/11925#discussion_r769803642



##########
File path: cpp/src/arrow/util/tracing_internal.cc
##########
@@ -53,6 +60,43 @@ namespace {
 
 namespace sdktrace = opentelemetry::sdk::trace;
 
+// Custom JSON stdout exporter. Leverages the OTLP HTTP exporter's
+// utilities to log the same format that would be sent to OTLP.

Review comment:
       Not totally understanding why a custom JSON format is less risky than 
using an upstream component or just having some documentation showing how to 
use the collector




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to