Skip to content

Geneva Exporter: Improve multithreaded support#3881

Merged
rajkumar-rangaraj merged 5 commits into
open-telemetry:mainfrom
mattsains:serialize-race
Feb 18, 2026
Merged

Geneva Exporter: Improve multithreaded support#3881
rajkumar-rangaraj merged 5 commits into
open-telemetry:mainfrom
mattsains:serialize-race

Conversation

@mattsains

Copy link
Copy Markdown
Contributor

Changes

Reviewed the trace and log exporter code and guarded against some potential race conditions.

Merge requirement checklist

  • CONTRIBUTING guidelines followed (license requirements, nullable enabled, static analysis, etc.)
  • Unit tests added/updated
  • Appropriate CHANGELOG.md files updated for non-trivial changes
  • Changes in public API reviewed (if applicable)

@github-actions github-actions Bot added the comp:exporter.geneva Things related to OpenTelemetry.Exporter.Geneva label Feb 18, 2026
@codecov

codecov Bot commented Feb 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.56%. Comparing base (c79d8a6) to head (eb205a1).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3881      +/-   ##
==========================================
+ Coverage   71.43%   71.56%   +0.13%     
==========================================
  Files         457      447      -10     
  Lines       17879    17827      -52     
==========================================
- Hits        12771    12758      -13     
+ Misses       5108     5069      -39     
Flag Coverage Δ
unittests-Exporter.Geneva 54.45% <100.00%> (-0.01%) ⬇️
unittests-Instrumentation.Cassandra ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...er.Geneva/Internal/MsgPack/MsgPackTraceExporter.cs 94.19% <100.00%> (-0.02%) ⬇️

... and 10 files with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mattsains
mattsains marked this pull request as ready for review February 18, 2026 19:49
@mattsains
mattsains requested a review from a team as a code owner February 18, 2026 19:49
Comment thread src/OpenTelemetry.Exporter.Geneva/CHANGELOG.md Outdated
Co-authored-by: Rajkumar Rangaraj <rajrang@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp:exporter.geneva Things related to OpenTelemetry.Exporter.Geneva

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants