Skip to content

Commit 01898be

Browse files
authored
perf: LogEnabled inline more (#2828)
1 parent e43f3df commit 01898be

File tree

3 files changed

+5
-2
lines changed

3 files changed

+5
-2
lines changed

opentelemetry-sdk/benches/log_enabled.rs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,8 @@
55
Total Number of Cores:   14 (10 performance and 4 efficiency)
66
| Test | Average time|
77
|---------------------------------------------|-------------|
8-
| exporter_disabled_concurrent_processor | 1.0 ns |
9-
| exporter_disabled_simple_processor | 4.5 ns |
8+
| exporter_disabled_concurrent_processor | 980 ps |
9+
| exporter_disabled_simple_processor | 4.3 ns |
1010
*/
1111

1212
// cargo bench --bench log_enabled --features="spec_unstable_logs_enabled,experimental_logs_concurrent_log_processor"
@@ -33,6 +33,7 @@ impl LogExporter for NoopExporter {
3333
Ok(())
3434
}
3535

36+
#[inline]
3637
fn event_enabled(
3738
&self,
3839
_level: opentelemetry::logs::Severity,

opentelemetry-sdk/src/logs/concurrent_log_processor.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,7 @@ impl<T: LogExporter> LogProcessor for SimpleConcurrentLogProcessor<T> {
4848
}
4949

5050
#[cfg(feature = "spec_unstable_logs_enabled")]
51+
#[inline]
5152
fn event_enabled(
5253
&self,
5354
level: opentelemetry::logs::Severity,

opentelemetry-sdk/src/logs/simple_log_processor.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -134,6 +134,7 @@ impl<T: LogExporter> LogProcessor for SimpleLogProcessor<T> {
134134
}
135135

136136
#[cfg(feature = "spec_unstable_logs_enabled")]
137+
#[inline]
137138
fn event_enabled(
138139
&self,
139140
level: opentelemetry::logs::Severity,

0 commit comments

Comments
 (0)