From 000906101f0b02ed25d139e80b73ce0bb08b45b1 Mon Sep 17 00:00:00 2001
From: Lalit Kumar Bhasin <lalit_fin@yahoo.com>
Date: Sat, 2 Nov 2024 10:57:40 -0700
Subject: [PATCH] review comments

---
 opentelemetry-sdk/src/trace/config.rs         | 12 ++++++------
 opentelemetry-sdk/src/trace/span_processor.rs |  2 +-
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/opentelemetry-sdk/src/trace/config.rs b/opentelemetry-sdk/src/trace/config.rs
index 94ba38bcff..b6ad2b150e 100644
--- a/opentelemetry-sdk/src/trace/config.rs
+++ b/opentelemetry-sdk/src/trace/config.rs
@@ -130,7 +130,7 @@ impl Default for Config {
                         Box::new(Sampler::TraceIdRatioBased(r))
                     } else {
                         otel_warn!(
-                            name: "TracerProvider.Config.Sampler.InvalidArgument",
+                            name: "TracerProvider.Config.InvalidSamplerArgument",
                             message = "OTEL_TRACES_SAMPLER is set to 'traceidratio' but OTEL_TRACES_SAMPLER_ARG environment variable is missing or invalid. OTEL_TRACES_SAMPLER_ARG must be a valid float between 0.0 and 1.0 representing the desired sampling probability (0.0 = no traces sampled, 1.0 = all traces sampled, 0.5 = 50% of traces sampled). Falling back to default ratio: 1.0 (100% sampling)",
                             otel_traces_sampler_arg = format!("{:?}", sampler_arg)
                         );
@@ -151,7 +151,7 @@ impl Default for Config {
                         ))))
                     } else {
                         otel_warn!(
-                            name: "TracerProvider.Config.Sampler.InvalidArgument",
+                            name: "TracerProvider.Config.InvalidSamplerArgument",
                             message = "OTEL_TRACES_SAMPLER is set to 'parentbased_traceidratio' but OTEL_TRACES_SAMPLER_ARG environment variable is missing or invalid. OTEL_TRACES_SAMPLER_ARG must be a valid float between 0.0 and 1.0 representing the desired sampling probability (0.0 = no traces sampled, 1.0 = all traces sampled, 0.5 = 50% of traces sampled). Falling back to default ratio: 1.0 (100% sampling)",
                             otel_traces_sampler_arg = format!("{:?}", sampler_arg)
                         );
@@ -162,28 +162,28 @@ impl Default for Config {
                 }
                 "parentbased_jaeger_remote" => {
                     otel_warn!(
-                        name: "TracerProvider.Config.Sampler.Unsupported",
+                        name: "TracerProvider.Config.UnsupportedSampler",
                         message = "OTEL_TRACES_SAMPLER is set to 'parentbased_jaeger_remote' which is not implemented in this SDK version. Using fallback sampler: ParentBased(AlwaysOn). Configure an alternative sampler using OTEL_TRACES_SAMPLER"
                     );
                     Box::new(Sampler::ParentBased(Box::new(Sampler::AlwaysOn)))
                 }
                 "jaeger_remote" => {
                     otel_warn!(
-                        name: "TracerProvider.Config.Sampler.Unsupported",
+                        name: "TracerProvider.Config.UnsupportedSampler",
                         message = "OTEL_TRACES_SAMPLER is set to 'jaeger_remote' which is implemented in this SDK version. Using fallback sampler: ParentBased(AlwaysOn). Configure an alternative sampler using OTEL_TRACES_SAMPLER"
                     );
                     Box::new(Sampler::ParentBased(Box::new(Sampler::AlwaysOn)))
                 }
                 "xray" => {
                     otel_warn!(
-                        name: "TracerProvider.Config.Sampler.Unsupported",
+                        name: "TracerProvider.Config.UnsupportedSampler",
                         message = "OTEL_TRACES_SAMPLER is set to 'xray'. AWS X-Ray sampler is not implemented in this SDK version. Using fallback sampler: ParentBased(AlwaysOn). Configure an alternative sampler using OTEL_TRACES_SAMPLER"
                     );
                     Box::new(Sampler::ParentBased(Box::new(Sampler::AlwaysOn)))
                 }
                 s => {
                     otel_warn!(
-                        name: "TracerProvider.Config.Sampler.InvalidSamplerType",
+                        name: "TracerProvider.Config.InvalidSamplerType",
                         message = format!(
                             "Unrecognized sampler type '{}' in OTEL_TRACES_SAMPLER environment variable. Valid values are: always_on, always_off, traceidratio, parentbased_always_on, parentbased_always_off, parentbased_traceidratio. Using fallback sampler: ParentBased(AlwaysOn)",
                             s
diff --git a/opentelemetry-sdk/src/trace/span_processor.rs b/opentelemetry-sdk/src/trace/span_processor.rs
index b490ff205c..9f26455bcc 100644
--- a/opentelemetry-sdk/src/trace/span_processor.rs
+++ b/opentelemetry-sdk/src/trace/span_processor.rs
@@ -251,7 +251,7 @@ impl<R: RuntimeChannel> SpanProcessor for BatchSpanProcessor<R> {
 
         if let Err(err) = result {
             otel_debug!(
-                name: "BatchSpanProcessor.Export.SendResultError",
+                name: "BatchSpanProcessor.ExportQueueingFailed",
                 reason = format!("{:?}", TraceError::Other(err.into()))
             );
         }