Skip to content

Conversation

@mstyura
Copy link
Contributor

@mstyura mstyura commented Feb 1, 2025

Fixes #
Design discussion issue (if applicable) #

Changes

Refactor code to prepare to work with pre-allocated and re-used buffer to serialize data exported to DD.

Merge requirement checklist

  • CONTRIBUTING guidelines followed
  • Unit tests added/updated (if applicable)
  • Appropriate CHANGELOG.md files updated for non-trivial, user-facing changes
  • Changes in public API reviewed (if applicable)
@codecov
Copy link

codecov bot commented Feb 1, 2025

Codecov Report

❌ Patch coverage is 92.85714% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 53.7%. Comparing base (5308844) to head (99dfc3c).
⚠️ Report is 191 commits behind head on main.

Files with missing lines Patch % Lines
opentelemetry-datadog/src/exporter/mod.rs 0.0% 4 Missing ⚠️
opentelemetry-datadog/src/exporter/model/v05.rs 94.1% 4 Missing ⚠️
opentelemetry-datadog/src/exporter/model/v03.rs 97.3% 1 Missing ⚠️
Additional details and impacted files
@@ Coverage Diff @@ ## main #162 +/- ## ======================================= + Coverage 53.6% 53.7% +0.1%  ======================================= Files 43 43 Lines 6166 6189 +23 ======================================= + Hits 3306 3327 +21  - Misses 2860 2862 +2 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
@mstyura mstyura force-pushed the serialize-into-writer-2 branch from a57572f to 445ba45 Compare February 1, 2025 14:59
@mstyura mstyura force-pushed the serialize-into-writer-2 branch from 445ba45 to 99dfc3c Compare February 1, 2025 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant