-
Notifications
You must be signed in to change notification settings - Fork 803
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
More OpenTelemetry changes #18246
More OpenTelemetry changes #18246
Conversation
|
Slightly less allocations. main:
this PR:
|
As with other OT changes, any Jaeger screenshots would be nice to have :) |
Hi @majocha would you like to add something else here or is it getting ready to merge? |
@psfinaki I think it's ok. Don't know how useful this will be in general, but it does reduce allocations a bit. |
/azp run |
Azure Pipelines successfully started running 2 pipeline(s). |
StackGuard.Guard
)Captured test run, some tests in parallel, a theory runs sequentially, Stackguard events with tags:

Traces from running VS instance:
