Lalit Kumar Bhasin

Results 436 comments of Lalit Kumar Bhasin
trafficstars

@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...