opentelemetry-rust
opentelemetry-rust copied to clipboard
open-telemetry
The Rust OpenTelemetry implementation
https://github.com/open-telemetry/opentelemetry-rust/blob/main/opentelemetry-sdk/src/logs/log_emitter.rs#L210 This clone needs some rethinking. Currently, every processor, even the ones that can stream event directly will have to suffer from this clone cost. I think we should just...
Benchmarks from [here](https://github.com/open-telemetry/opentelemetry-rust/blob/main/opentelemetry/benches/attributes.rs) shows that the cost of creating abstractions like Key,Value in `opentelemetry` crate is non-trivial. Take the example below: ``` c.bench_function("CreateOTelKeyValue", |b| { b.iter(|| { let _v1 =...
### Related Problems? The MeterProvider shutdown sequence is `SdkMeterProvider ::Shutdown()` -> `opentelemetry_sdk::metrics::Pipeline::shutdown()` -> `PeriodicReader::shutdown()` -> `PushMetricsExporter::shutdown` So on shutdown, - the MetricReader will do one last collect-and-export cycle, and then...
not really introduced in this PR, but the API exposing things like `dropped_attributes_count` is not good :( It is up-to the SDK to enforce the limit and store the count...
Ran into an interesting edge case when trying to put together a sample for #1337 It seems like they're using OpenTelemetryOTLP exporting. However that's not the main topic here. We...
Fixes #1896 Please provide a brief description of the changes here. ## Merge requirement checklist * [x] [CONTRIBUTING](https://github.com/open-telemetry/opentelemetry-rust/blob/main/CONTRIBUTING.md) guidelines followed * [ ] Unit tests added/updated (if applicable) * [...
As discussed in linked issue, There is a noticeable performance degradation when using the [AnyValue ](https://github.com/open-telemetry/opentelemetry-rust/blob/d5c86d9adc6f0e3f2612dcfd92b8b64e88ab20d7/opentelemetry/src/logs/record.rs#L51) enum compared to the [Value](https://github.com/open-telemetry/opentelemetry-rust/blob/d5c86d9adc6f0e3f2612dcfd92b8b64e88ab20d7/opentelemetry/src/common.rs#L243) enum. This needs to be investigated further. _Originally posted...
## Changes - Update metrics benchmarks for API - Use `iter_batched` method to generate the dynamic attributes before each iteration - Add `black_box` calls to existing benchmark method
 Since we don't pre-allocate, the vec starts with size 1, and often requires re-allocating. Would we start with a default size of, say 5? (don't ask me why 5,...
## Changes As discussed [here](https://github.com/open-telemetry/opentelemetry-rust/pull/1854#discussion_r1623663493) Make `SetResource` return status, and call it blockng in Batch[Log|Span]Processor. This makes the setting of resources during initialization deterministic. Earlier ```rust LogExporter::set_resource(&mut self, resource: &opentelemetry_sdk::Resource);...