-
Notifications
You must be signed in to change notification settings - Fork 385
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
sdk: optimize span creation internals #693
Conversation
* switch from `Option::map` to pattern match in sdk tracer: ~16% perf improvement on span creation. * optimize for common case of single span processor in sdk span: ~1% perf improvement on span end Overall improvement on span start-end benchmark: ~17%
Codecov Report
@@ Coverage Diff @@
## main #693 +/- ##
==========================================
- Coverage 71.41% 71.39% -0.03%
==========================================
Files 101 101
Lines 8551 8561 +10
==========================================
+ Hits 6107 6112 +5
- Misses 2444 2449 +5
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spent some more time looking on this, more style suggestions:
Option::map
to pattern match in sdk tracer: ~16% perf improvement on span creation.Overall improvement on span start-end benchmark: ~17%