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

Add metrics aggregation tests to ensure correctness

Open cijothomas opened this issue 1 year ago • 2 comments

The current tests are only covering Counters.

We need to replicate the same for all instruments. Multi-threaded tests are nice to have, but not a must before beta.

cijothomas avatar May 24 '24 00:05 cijothomas

I would like to take this up.

utpilla avatar May 24 '24 00:05 utpilla

Thanks! Assigned. Lack of test is giving me less confidence to do big refactoring to get perf boost. For now I'll focus on Counters, which do have some coverage.

cijothomas avatar May 24 '24 00:05 cijothomas

Closing as we have this mostly covered.

cijothomas avatar Nov 23 '24 03:11 cijothomas