open-telemetry
/
opentelemetry-dotnet
mirror of https://github.com/open-telemetry/opentelemetry-dotnet.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix localization issue for Probabilistic Sampler description (#700)

This commit is contained in:
Paulo Janotti 2020-05-26 09:51:47 -07:00 committed by GitHub
parent 4aefc62df8
commit acaec5d9ab
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/OpenTelemetry/Trace/Samplers/ProbabilitySampler.cs
View File

@ -16,6 +16,7 @@
using System;
using System.Collections.Generic;
using System.Diagnostics;
using System.Globalization;
namespace OpenTelemetry.Trace.Samplers
{
@ -46,7 +47,7 @@ namespace OpenTelemetry.Trace.Samplers
// The expected description is like ProbabilitySampler{0.000100}
// https://docs.microsoft.com/dotnet/standard/base-types/composite-formatting#escaping-braces
this.Description = string.Format("{0}{1:F6}{2}", "ProbabilitySampler{", this.probability, "}");
this.Description = "ProbabilitySampler{" + this.probability.ToString("F6", CultureInfo.InvariantCulture) + "}";
// Special case the limits, to avoid any possible issues with lack of precision across
// double/long boundaries. For probability == 0.0, we use Long.MIN_VALUE as this guarantees