Review comment

lalitb · lalitb · commit 27558d9b31c5 · 2024-10-27T21:47:45.000-07:00
diff --git a/opentelemetry-sdk/src/trace/sampler/jaeger_remote/sampling_strategy.rs b/opentelemetry-sdk/src/trace/sampler/jaeger_remote/sampling_strategy.rs
@@ -107,7 +107,7 @@ impl Inner {
                 }
             })
             .unwrap_or_else(|_err| {
-                otel_debug!(
+                otel_warn!(
                     name: "JaegerRemoteSampler.MutexPoisoned",
                     message = "Failed to update Jaeger Remote sampling strategy. The sampler's internal mutex is poisoned, indicating a panic occurred in another thread holding the lock. Sampling decisions may be using stale configuration.",
                 );

Original file line number	Diff line number	Diff line change
`@@ -107,7 +107,7 @@ impl Inner {`
`107`	`107`	`}`
`108`	`108`	`})`
`109`	`109`	`.unwrap_or_else(\|_err\| {`
`110`		`- otel_debug!(`
	`110`	`+ otel_warn!(`
`111`	`111`	`name: "JaegerRemoteSampler.MutexPoisoned",`
`112`	`112`	`message = "Failed to update Jaeger Remote sampling strategy. The sampler's internal mutex is poisoned, indicating a panic occurred in another thread holding the lock. Sampling decisions may be using stale configuration.",`
`113`	`113`	`);`