fix(rc): handle null values in RC APM Tracing payloads by dmehala · Pull Request #236 · DataDog/dd-trace-cpp

@dmehala

Description

Ensures system-tests pass by properly ignoring null values APM Tracing RC payload.

@codecov-commenter

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.45%. Comparing base (c6185cf) to head (69dc58e).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #236   +/-   ##
=======================================
  Coverage   87.44%   87.45%           
=======================================
  Files          83       83           
  Lines        5458     5460    +2     
=======================================
+ Hits         4773     4775    +2     
  Misses        685      685           

☔ 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.

@dmehala

Ensures system-tests pass by properly ignoring null values APM
Tracing RC payload.

@pr-commenter

Benchmarks

Benchmark execution time: 2025-09-04 12:29:02

Comparing candidate commit 69dc58e in PR branch dmehala/rc-apm-tracing-ignore-null with baseline commit c6185cf in branch main.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 1 metrics, 0 unstable metrics.

dubloom

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dmehala dmehala deleted the dmehala/rc-apm-tracing-ignore-null branch

September 4, 2025 12:43