Skip to content

Remove zero duration span from slow rendering instrumentation.#1729

Merged
breedx-splk merged 2 commits into
open-telemetry:mainfrom
breedx-splk:zero_duration_spans_to_events
May 11, 2026
Merged

Remove zero duration span from slow rendering instrumentation.#1729
breedx-splk merged 2 commits into
open-telemetry:mainfrom
breedx-splk:zero_duration_spans_to_events

Conversation

@breedx-splk
Copy link
Copy Markdown
Contributor

We deprecated the old zero-duration span compatibility 8+ months ago, so it's long overdue for removal. Events are the way.

@breedx-splk breedx-splk requested a review from a team as a code owner May 8, 2026 20:07
@codecov
Copy link
Copy Markdown

codecov Bot commented May 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.06%. Comparing base (68771b5) to head (bef2a09).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1729      +/-   ##
==========================================
- Coverage   62.46%   62.06%   -0.40%     
==========================================
  Files         160      159       -1     
  Lines        3479     3443      -36     
  Branches      353      347       -6     
==========================================
- Hits         2173     2137      -36     
+ Misses       1208     1207       -1     
- Partials       98       99       +1     

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

Copy link
Copy Markdown
Member

@fractalwrench fractalwrench left a comment

Choose a reason for hiding this comment

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

LGTM

@breedx-splk breedx-splk force-pushed the zero_duration_spans_to_events branch from a603d6f to bef2a09 Compare May 11, 2026 16:50
@breedx-splk breedx-splk enabled auto-merge (squash) May 11, 2026 16:50
@breedx-splk breedx-splk merged commit 00fa630 into open-telemetry:main May 11, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants