birschick-bq commented on code in PR #3315: URL: https://github.com/apache/arrow-adbc/pull/3315#discussion_r2328813619
########## csharp/src/Telemetry/Traces/Exporters/FileExporter/SerializableActivityJsonContext.cs: ########## Review Comment: @CurtHagenlocher - Just checking to see if there is something that needs to be addressed before we add exporters. Can you let me know any of the following are important to look at? 1. Performance/stability of the file exporter? 2. Only add the file exporter on the drivers and not the OTel-based exporters? 3. Handle the ability to start and stop file tracing, connection-by-connection, rather than a global static. -- 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]
