Skip to content

Conversation

dmitryax
Copy link
Member

@dmitryax dmitryax commented Jun 17, 2025

Resolves #13220 (comment)

Benchmarks:

goos: darwin
goarch: arm64
pkg: go.opentelemetry.io/collector/pdata/xpdata/request
cpu: Apple M1 Max

before:
BenchmarkEncodeDecodeContext-10    	  528876	      2331 ns/op	    1200 B/op	      28 allocs/op

after:
BenchmarkEncodeDecodeContext-10    	  559353	      1832 ns/op	     864 B/op	      16 allocs/op

@dmitryax dmitryax requested review from a team and bogdandrutu as code owners June 17, 2025 21:53
@dmitryax dmitryax force-pushed the change-span-context-serialization branch 2 times, most recently from a12b772 to 4fa7dc6 Compare June 17, 2025 22:04
Copy link

codecov bot commented Jun 17, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.50%. Comparing base (c779925) to head (3bf8428).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #13230      +/-   ##
==========================================
- Coverage   91.51%   91.50%   -0.02%     
==========================================
  Files         522      522              
  Lines       28980    28988       +8     
==========================================
+ Hits        26522    26525       +3     
- Misses       1940     1942       +2     
- Partials      518      521       +3     

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

@dmitryax dmitryax force-pushed the change-span-context-serialization branch from 4fa7dc6 to 3bf8428 Compare June 17, 2025 22:21
@dmitryax dmitryax added this pull request to the merge queue Jun 17, 2025
Merged via the queue into open-telemetry:main with commit 8d71881 Jun 18, 2025
56 checks passed
@dmitryax dmitryax deleted the change-span-context-serialization branch June 18, 2025 00:20
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