Skip to content

Commit

Permalink
Revert "Add InstrumentationLibrary to Sampler.ShouldSample (#1850)" (#…
Browse files Browse the repository at this point in the history
…1942)

This reverts commit f936f2e.

Co-authored-by: Bogdan Drutu <[email protected]>
  • Loading branch information
iNikem and bogdandrutu authored Sep 22, 2021
1 parent e0fb920 commit 6d44e2e
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 4 deletions.
2 changes: 0 additions & 2 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,6 @@ release.

- Prefer global user defined limits over model-sepcific default values.
([#1893](https://github.com/open-telemetry/opentelemetry-specification/pull/1893))
- Add InstrumentationLibrary to Sampler.ShouldSample.
([#1850](https://github.com/open-telemetry/opentelemetry-specification/pull/1850))

### Metrics

Expand Down
1 change: 0 additions & 1 deletion spec-compliance-matrix.md
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,6 @@ formats is required. Implementing more than one format is optional.
| [Sampling](specification/trace/sdk.md#sampling) | | | | | | | | | | | | |
| Allow samplers to modify tracestate | | + | + | | + | + | + | | + | + | - | + |
| ShouldSample gets full parent Context | | + | + | + | + | + | + | | + | + | - | + |
| ShouldSample gets InstrumentationLibrary | | | | | | | | | | | | |
| [New Span ID created also for non-recording Spans](specification/trace/sdk.md#sdk-span-creation) | | + | + | | + | + | + | | + | + | - | + |
| [IdGenerators](specification/trace/sdk.md#id-generators) | | + | + | | + | + | | | + | + | | + |
| [SpanLimits](specification/trace/sdk.md#span-limits) | X | + | + | | + | + | | | | - | | + |
Expand Down
1 change: 0 additions & 1 deletion specification/trace/sdk.md
Original file line number Diff line number Diff line change
Expand Up @@ -210,7 +210,6 @@ Returns the sampling Decision for a `Span` to be created.
* Collection of links that will be associated with the `Span` to be created.
Typically useful for batch operations, see
[Links Between Spans](../overview.md#links-between-spans).
* `InstrumentationLibrary` (name and version) of the `Span` to be created.

Note: Implementations may "bundle" all or several arguments together in a single
object.
Expand Down

0 comments on commit 6d44e2e

Please sign in to comment.