Skip to content

Commit

Permalink
fix build error
Browse files Browse the repository at this point in the history
  • Loading branch information
lalitb committed Oct 28, 2024
1 parent 182cd2c commit 12b00eb
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ use crate::trace::{Sampler, ShouldSample};
use futures_util::{stream, StreamExt as _};
use http::Uri;
use opentelemetry::trace::{Link, SamplingResult, SpanKind, TraceError, TraceId};
use opentelemetry::{otel_error, Context, KeyValue};
use opentelemetry::{otel_warn, Context, KeyValue};
use opentelemetry_http::HttpClient;
use std::str::FromStr;
use std::sync::Arc;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -142,10 +142,8 @@ impl Inner {
otel_warn!(
name: "Sampler.JaegerRemote.InvalidStrategy",
message = "Received invalid sampling strategy from Jaeger remote endpoint. Expected one of: OperationSampling, RateLimitingSampling (max traces per second), or ProbabilisticSampling (0.0-1.0 sampling probability). No valid strategy was found in the response. Using previous strategy if available.",
received_operation_sampling = operation_sampling.is_some(),
received_rate_limiting = rate_limiting_sampling.is_some(),
received_probabilistic = probabilistic_sampling.is_some()
);
None

Check warning on line 146 in opentelemetry-sdk/src/trace/sampler/jaeger_remote/sampling_strategy.rs

View check run for this annotation

Codecov / codecov/patch

opentelemetry-sdk/src/trace/sampler/jaeger_remote/sampling_strategy.rs#L142-L146

Added lines #L142 - L146 were not covered by tests
}
}
}
Expand Down

0 comments on commit 12b00eb

Please sign in to comment.