Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[exporter/azuremonitor] Refactor to use higher level api to send trace events #35655

Closed

Conversation

whitneygriffith
Copy link
Contributor

@whitneygriffith whitneygriffith commented Oct 7, 2024

Description

  1. Created the spanToTelemetry method to leverage the higher-level Track API, which is the recommended approach for sending traces to the queue. This refactor simplifies trace processing and aligns with best practices for handling telemetry data efficiently.
  2. Resolved an issue where traces weren't being sent to App Insights due to not flushing the Telemetry Client. Added the necessary flush operation to ensure all traces are properly sent to the queue, leveraging App Insights' batch handling for more efficient processing.

Link to tracking issue

Part of #35037

I will follow up with updating the API used for sending metrics and logs as well. When this is done I will deprecate the transportChannel which uses the lower level Send() method

Signed-off-by: whitneygriffith <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants