Cijo Thomas

What should provider's force_flush return

1

We can return an overall Result Return a Vec of results from each processor/reader. We need to be consistent across signals, unless there is a good reason to deviate. originally...

OTLP Exporter must default to http/protobuf

1

The current default in OTLP Exporter is grpc : https://github.com/open-telemetry/opentelemetry-rust/blob/main/opentelemetry-otlp/Cargo.toml#L64 This must be changed to use http/protobuf, as perf https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/protocol/exporter.md#specify-protocol

SDK/Export concepts leaked into API.

3

https://github.com/open-telemetry/opentelemetry-rust/blob/main/opentelemetry/src/common.rs#L426 https://github.com/open-telemetry/opentelemetry-rust/blob/main/opentelemetry/src/logs/mod.rs#L27 The above are part of Public API of the OpenTelemetry-API create. Its about "Export" which is an SDK only concept. Wondering if this is intentional or accidental? If...

Metric - concurrency test improvements

1

Current tests are very limited: https://github.com/open-telemetry/opentelemetry-dotnet/blob/main/test/OpenTelemetry.Tests/Metrics/MetricAPITest.cs#L241 1. Need to test the special case of "no tags". 2. Need to add test where different tags are updated by diff. threads. (To...

Concurrent sending in OTLP exporter

1

Opening to clarify "Concurrent sending" in spec compliance matrix for OTLP exporter. https://github.com/open-telemetry/opentelemetry-specification/blob/main/spec-compliance-matrix.md#exporters The following describes concurrent requests: https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/protocol/otlp.md#otlpgrpc-concurrent-requests But the Exporter spec (https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/trace/sdk.md#exportbatch) says "_Export() will never be called...

SpanProcessor.OnStart, Sampler receives parent Context

2

Spec requires SpanProcesor.OnStart to receive the span and parentcontext (https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/trace/sdk.md#onstart). The .NET Implementation only provides Activity to OnStart on ActivityProcessor, and does not explicitly provide a parent `Context`. However, it...

Metrics hot path now involves copying the [incoming ReadOnlySpan into a Thread-Local array](https://github.com/open-telemetry/opentelemetry-dotnet/blob/main/src/OpenTelemetry/Metrics/AggregatorStore.cs#L1087-L1100) to do the dictionary lookup. With the new capability to [lookup Dictionary with just ReadOnlySpan](https://x.com/davidfowl/status/1821070606609084696), we can...

Avoid string copy in tracing appender

3

Tracing appender's EventVisitor currently copies the [`&str` to an owned string](https://github.com/open-telemetry/opentelemetry-rust/blob/main/opentelemetry-appender-tracing/src/layer.rs#L102) in the heap. This is the case even when the user provides 'static str to the tracing macros. Opening...

OTel Metrics with no heap allocation

3

Opening a parent issue to track making it possible to do OTel Metrics with zero heap allocation. Since Metrics are always pre-aggregated and exported once every few seconds in background,...

Should Exporter force async?

4

Exporter traits are [now written as async](https://github.com/open-telemetry/opentelemetry-rust/blob/main/opentelemetry-sdk/src/export/logs/mod.rs#L15-L18). This works great if the exporter actually needs to use async. But when exporter is not needing async (like when exporting to kernel...