opentelemetry-dotnet icon indicating copy to clipboard operation
opentelemetry-dotnet copied to clipboard

Published 20 hours ago • verified publisher icon open-telemetry

Perf improvement using zero-alloc foreach

Open reyang opened this issue 5 years ago • 2 comments

https://github.com/open-telemetry/opentelemetry-dotnet/pull/1078#discussion_r470401687

Potential places that we want to improve:

  1. Processor chain (CompositeActivityProcessor).
  2. Exporter interface (a batch of spans).
  3. Resource attributes.

reyang avatar Aug 14 '20 05:08 reyang

@CodeBlanch has an idea of using union approach to address the IEnumerator improvement among Processor/CircularBuffer/Exporter.

reyang avatar Aug 18 '20 00:08 reyang

There's been a fix related to this, if there's more work I'd suggest a more targeted suggestion.

martinjt avatar Dec 12 '23 18:12 martinjt

Most of the work has completed, if there's more work then you can reopen with the specific suggestions and why, or create a new targeted ticket for them.

I think there's been a tonne of work around allocations in the project, so it's probably something that has already been done. So I'm going to mark this as complete.

martinjt avatar Feb 22 '24 13:02 martinjt