-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
chore: remove unconditioned instrument code #11237
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
base: main
Are you sure you want to change the base?
Conversation
|
|
It has a noticeable cost. You can try enabling |
f5721a4 to
ecd55a6
Compare
CodSpeed Performance ReportMerging #11237 will not alter performanceComparing Summary
|
|
In rspack-resolver, when I enable |
Binary Sizes
Commit: f4fda6e |
|
I think it's a good idea to remove all tracing code, considering the perf difference |
|
According to codspeed's report, the most improved step is enabling |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub. |
To check if tracing has a performance cost
Description:
BREAKING CHANGE:
Related issue (if exists):