Skip to content

Commit e43f3df

Browse files
authored
perf: Inline enabled check in Logger (#2823)
1 parent 40effae commit e43f3df

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

opentelemetry-sdk/benches/log_enabled.rs

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

12+
// cargo bench --bench log_enabled --features="spec_unstable_logs_enabled,experimental_logs_concurrent_log_processor"
13+
1214
use criterion::{criterion_group, criterion_main, Criterion};
1315
use opentelemetry::logs::{Logger, LoggerProvider};
1416
use opentelemetry_sdk::error::OTelSdkResult;

opentelemetry-sdk/src/logs/logger.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,7 @@ impl opentelemetry::logs::Logger for SdkLogger {
5050
}
5151

5252
#[cfg(feature = "spec_unstable_logs_enabled")]
53+
#[inline]
5354
fn event_enabled(&self, level: Severity, target: &str, name: Option<&str>) -> bool {
5455
self.provider
5556
.log_processors()

0 commit comments

Comments
 (0)