Skip to content

Commit b18a0f8

Browse files
make IntegrationSettings.AnalyticsSampleRate not nullable and default to 1.0 (DataDog#324)
1 parent 29d453b commit b18a0f8

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

src/Datadog.Trace/Configuration/IntegrationSettings.cs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,8 @@ public IntegrationSettings(string integrationName, IConfigurationSource source)
2525

2626
AnalyticsEnabled = source.GetBool(string.Format(ConfigurationKeys.Integrations.AnalyticsEnabled, integrationName));
2727

28-
AnalyticsSampleRate = source.GetDouble(string.Format(ConfigurationKeys.Integrations.AnalyticsSampleRate, integrationName));
28+
AnalyticsSampleRate = source.GetDouble(string.Format(ConfigurationKeys.Integrations.AnalyticsSampleRate, integrationName)) ??
29+
1.0;
2930
}
3031

3132
/// <summary>
@@ -49,6 +50,6 @@ public IntegrationSettings(string integrationName, IConfigurationSource source)
4950
/// Gets or sets a value between 0 and 1 (inclusive)
5051
/// that determines the sampling rate for this integration.
5152
/// </summary>
52-
public double? AnalyticsSampleRate { get; set; }
53+
public double AnalyticsSampleRate { get; set; }
5354
}
5455
}

src/Datadog.Trace/Configuration/TracerSettings.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -183,7 +183,7 @@ internal bool IsIntegrationEnabled(string name)
183183
{
184184
var integrationSettings = Integrations[name];
185185
var analyticsEnabled = integrationSettings.AnalyticsEnabled ?? (enabledWithGlobalSetting && AnalyticsEnabled);
186-
return analyticsEnabled ? integrationSettings.AnalyticsSampleRate : null;
186+
return analyticsEnabled ? integrationSettings.AnalyticsSampleRate : (double?)null;
187187
}
188188
}
189189
}

0 commit comments

Comments
 (0)