Christian Neumüller

A span with a new span ID MUST be created even for a sampling decision of DROP.

13

# Bug Report This bug should be present in master and at least since [0.5.0-beta](https://github.com/open-telemetry/opentelemetry-dotnet/releases/tag/0.5.0-beta) (probably since the beginning) ## Symptom If the sampler returns a decision of DROP, StartActivity...

Check if we should preserve more of the Context in the OpenTracing shim.

2

Currently when a span is created in SpanBuilderShim.start, we only have the parent OpenTracing spanContext available. This means that we lose the originial full context when calling SpanProcessor.OnStart (not yet...

Instrumentation library info and Resources are not available to Samplers

25

ShouldSample receives as input all the known information about the Span to be created, see https://github.com/open-telemetry/opentelemetry-specification/blob/v1.1.0/specification/trace/sdk.md#shouldsample, *except* the instrumentation library info and the Resources. Suppressing unwanted spans depending on the...

Consider adding a resource (semantic convention) to distinguish HTTP applications (http.app attribute)

8

See https://github.com/open-telemetry/opentelemetry-specification/pull/263#issuecomment-544441748 and https://github.com/open-telemetry/opentelemetry-specification/pull/263#discussion_r329281769. We should find a way to represent information about the HTTP application (proposed name semantic attribute name `http.app`) *somewhere*, ideally somewhere where for each incoming HTTP...

Propose Context-scoped attributes.

53

**Add Context-scoped telemetry attributes which typically apply to all signals associated with a trace as it crosses a single service.** This OTEP aims to address various related demands that have...

Labels for closed + unmerged PRs

3

The OTEP process defines the states `rejected` and `deferred` which lead to closing a PR without it being merged. It would be great if we had & assigned labels to...

Commit a3d23cc12df52cb10ee0102e50cd54eb943527e6 ensured that the thread context class loader is set when invoking the `service` method. However, the context class loader is currently still not set when invoking newInstance, causing,...

Review approach & specify algorithm for TraceIdRatioBasedSampler (ProbabilitySampler)

25

See also discussion on https://github.com/open-telemetry/opentelemetry-specification/pull/1412#issuecomment-776101910 The sampling alorithm for TraceIdRatioBasedSampler is unspecified. As a result, trace IDs that are sampled by some implementations might get non-sampled or re-sampled by SDKs...

Investigate if we can/want to make fields and return values @Nonnull by default.

1

https://stackoverflow.com/a/11807961/2128694 suggests there is a way to annotate a package so that fields and/or return values are `@Nonnull` by default, similarly to the `@ParametersAreNonnullByDefault` we already use. This answer is...

sf::String <-> ANSI string conversion is broken.

13

## Symptoms ### Steps to reproduce 1. Create a `std::locale` which uses an double byte character encoding. 2. Construct a `sf::String` from a string/character literal which contains characters represented as...