Skip to content

Commit ecd8a21

Browse files
committed
remove simple log processor
1 parent 406e31b commit ecd8a21

File tree

1 file changed

+41
-5
lines changed
  • opentelemetry-sdk/benches

1 file changed

+41
-5
lines changed

opentelemetry-sdk/benches/log.rs

+41-5
Original file line numberDiff line numberDiff line change
@@ -16,26 +16,62 @@ use opentelemetry::trace::Tracer;
1616
use opentelemetry::trace::TracerProvider as _;
1717
use opentelemetry::Key;
1818
use opentelemetry_sdk::export::logs::{LogData, LogExporter};
19+
use opentelemetry_sdk::logs::LogProcessor;
1920
use opentelemetry_sdk::logs::{Logger, LoggerProvider};
2021
use opentelemetry_sdk::trace;
2122
use opentelemetry_sdk::trace::{Sampler, TracerProvider};
2223

2324
#[derive(Debug)]
24-
struct VoidExporter;
25+
struct NoopExporter;
2526

2627
#[async_trait]
27-
impl LogExporter for VoidExporter {
28-
async fn export<'a>(&mut self, _batch: Vec<std::borrow::Cow<'a, LogData>>) -> LogResult<()> {
28+
impl LogExporter for NoopExporter {
29+
async fn export<'a>(&mut self, _: Vec<std::borrow::Cow<'a, LogData>>) -> LogResult<()> {
2930
LogResult::Ok(())
3031
}
3132
}
3233

34+
#[derive(Debug)]
35+
struct NoopProcessor {
36+
exporter: Box<dyn LogExporter>,
37+
}
38+
39+
impl NoopProcessor {
40+
fn new(exporter: Box<dyn LogExporter>) -> Self {
41+
Self { exporter }
42+
}
43+
}
44+
45+
impl LogProcessor for NoopProcessor {
46+
fn emit(&self, _: &mut LogData) {
47+
// no-op
48+
}
49+
50+
fn force_flush(&self) -> LogResult<()> {
51+
Ok(())
52+
}
53+
54+
fn shutdown(&self) -> LogResult<()> {
55+
Ok(())
56+
}
57+
58+
#[cfg(feature = "logs_level_enabled")]
59+
fn event_enabled(
60+
&self,
61+
_level: opentelemetry::logs::Severity,
62+
_target: &str,
63+
_name: &str,
64+
) -> bool {
65+
true
66+
}
67+
}
68+
3369
fn log_benchmark_group<F: Fn(&Logger)>(c: &mut Criterion, name: &str, f: F) {
3470
let mut group = c.benchmark_group(name);
3571

3672
group.bench_function("no-context", |b| {
3773
let provider = LoggerProvider::builder()
38-
.with_simple_exporter(VoidExporter)
74+
.with_log_processor(NoopProcessor::new(Box::new(NoopExporter)))
3975
.build();
4076

4177
let logger = provider.logger("no-context");
@@ -45,7 +81,7 @@ fn log_benchmark_group<F: Fn(&Logger)>(c: &mut Criterion, name: &str, f: F) {
4581

4682
group.bench_function("with-context", |b| {
4783
let provider = LoggerProvider::builder()
48-
.with_simple_exporter(VoidExporter)
84+
.with_log_processor(NoopProcessor::new(Box::new(NoopExporter)))
4985
.build();
5086

5187
let logger = provider.logger("with-context");

0 commit comments

Comments
 (0)