Lalit Kumar Bhasin
Lalit Kumar Bhasin
@krlohnes Thanks for the PR. Changes look good, but you need to sign the CLA before it can be merged.
Thanks, let's track this potential improvement with benchmark of both memory and throughput while handling this.
I think we discussed it earlier - If we plan to start with default size, we can also evaluate using [smallvec](https://crates.io/crates/smallvec), to allocate this initial capacity on stack. [tokio-tracing](https://github.com/tokio-rs/tracing-opentelemetry/blob/9462de6481321704912f54d3f61c080eb732ca62/src/metrics.rs#L177) is...
Other option - https://github.com/microsoft/cpp_client_telemetry/pull/336 Also related issue - https://github.com/microsoft/cpp_client_telemetry/issues/652
Thanks @lakshmir124, can you please add more details to the PR description?
PR branch: ```bash $ cargo bench --bench context_suppression Finished `bench` profile [optimized + debuginfo] target(s) in 0.13s Running benches/context_suppression.rs (/tmp/tbd_a/opentelemetry-rust/target/release/deps/context_suppression-3de0ed5c3fd35dc6) Gnuplot not found, using plotters backend telemetry_suppression/enter_telemetry_suppressed_scope time: [26.002 ns...