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
fix: Tracing span names should match function names #17148
Conversation
it's also kinda more annoying if you only want to record some parameters. The runtime speed should be the same, it's just a papercut in compile times. |
Fwiw, using the attribute is also worse for rust-analyzer analysis performance |
@bors delegate+ |
When viewing traces, it's slightly confusing when the span name doesn't match the function name. Ensure the names are consistent. (It might be worth moving most of these to use #[tracing::instrument] so the name can never go stale. @davidbarsky suggested that is marginally slower, so I've just done the simple change here.)
☀️ Test successful - checks-actions |
When viewing traces, it's slightly confusing when the span name doesn't match the function name. Ensure the names are consistent.
(It might be worth moving most of these to use
#[tracing::instrument]
so the name can never go stale. @davidbarsky suggested that is marginally slower, so I've just done the simple change here.)