Skip to content

Commit ee26cbb

Browse files
committed
SamplingResult constructor accepts null parameters
1 parent 5aa077b commit ee26cbb

File tree

3 files changed

+13
-3
lines changed

3 files changed

+13
-3
lines changed

src/OpenTelemetry/.publicApi/Stable/PublicAPI.Unshipped.txt

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,4 +25,8 @@ OpenTelemetry.ReadOnlyFilteredTagCollection.Enumerator.Enumerator() -> void
2525
OpenTelemetry.ReadOnlyFilteredTagCollection.Enumerator.MoveNext() -> bool
2626
OpenTelemetry.ReadOnlyFilteredTagCollection.GetEnumerator() -> OpenTelemetry.ReadOnlyFilteredTagCollection.Enumerator
2727
OpenTelemetry.ReadOnlyFilteredTagCollection.ReadOnlyFilteredTagCollection() -> void
28+
*REMOVED*OpenTelemetry.Trace.SamplingResult.SamplingResult(OpenTelemetry.Trace.SamplingDecision decision, string! traceStateString) -> void
29+
*REMOVED*OpenTelemetry.Trace.SamplingResult.SamplingResult(OpenTelemetry.Trace.SamplingDecision decision, System.Collections.Generic.IEnumerable<System.Collections.Generic.KeyValuePair<string!, object!>>! attributes) -> void
30+
OpenTelemetry.Trace.SamplingResult.SamplingResult(OpenTelemetry.Trace.SamplingDecision decision, string? traceStateString) -> void
31+
OpenTelemetry.Trace.SamplingResult.SamplingResult(OpenTelemetry.Trace.SamplingDecision decision, System.Collections.Generic.IEnumerable<System.Collections.Generic.KeyValuePair<string!, object!>>? attributes) -> void
2832
static OpenTelemetry.Metrics.MeterProviderBuilderExtensions.SetExemplarFilter(this OpenTelemetry.Metrics.MeterProviderBuilder! meterProviderBuilder, OpenTelemetry.Metrics.ExemplarFilterType exemplarFilter) -> OpenTelemetry.Metrics.MeterProviderBuilder!

src/OpenTelemetry/CHANGELOG.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,12 @@
2525
supported in stable builds.
2626
([#5607](https://github.com/open-telemetry/opentelemetry-dotnet/pull/5607))
2727

28+
* `SamplingResult(SamplingDecision decision,
29+
IEnumerable<KeyValuePair<string, object>>? attributes)`
30+
and `SamplingResult(SamplingDecision decision, string? traceStateString)` constructors
31+
accept `null` for `attributes` and `traceStateString` parameters respectively.
32+
([#5614](https://github.com/open-telemetry/opentelemetry-dotnet/pull/5614))
33+
2834
## 1.8.1
2935

3036
Released 2024-Apr-17

src/OpenTelemetry/Trace/SamplingResult.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ public SamplingResult(bool isSampled)
3232
/// <param name="decision">indicates whether an activity object is recorded and sampled.</param>
3333
/// <param name="attributes">Attributes associated with the sampling decision. Attributes list passed to
3434
/// this method must be immutable. Mutations of the collection and/or attribute values may lead to unexpected behavior.</param>
35-
public SamplingResult(SamplingDecision decision, IEnumerable<KeyValuePair<string, object>> attributes)
35+
public SamplingResult(SamplingDecision decision, IEnumerable<KeyValuePair<string, object>>? attributes)
3636
: this(decision, attributes, traceStateString: null)
3737
{
3838
}
@@ -42,7 +42,7 @@ public SamplingResult(SamplingDecision decision, IEnumerable<KeyValuePair<string
4242
/// </summary>
4343
/// <param name="decision">indicates whether an activity object is recorded and sampled.</param>
4444
/// <param name="traceStateString">traceStateString associated with the created Activity.</param>
45-
public SamplingResult(SamplingDecision decision, string traceStateString)
45+
public SamplingResult(SamplingDecision decision, string? traceStateString)
4646
: this(decision, attributes: null, traceStateString)
4747
{
4848
}
@@ -67,7 +67,7 @@ public SamplingResult(SamplingDecision decision, IEnumerable<KeyValuePair<string
6767
}
6868

6969
/// <summary>
70-
/// Gets a value indicating indicates whether an activity object is recorded and sampled.
70+
/// Gets a value indicating whether an activity object is recorded and sampled.
7171
/// </summary>
7272
public SamplingDecision Decision { get; }
7373

0 commit comments

Comments
 (0)