Skip to content
Merged
Changes from 3 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions opentelemetry-sdk/src/metrics/manual_reader.rs
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,8 @@
};

use opentelemetry::{
global,
metrics::{MetricsError, Result},
otel_debug,
};

use super::{
Expand Down Expand Up @@ -77,9 +77,9 @@
if inner.sdk_producer.is_none() {
inner.sdk_producer = Some(pipeline);
} else {
global::handle_error(MetricsError::Config(
"duplicate reader registration, did not register manual reader".into(),
))
otel_debug!(
name: "ManualReader.DuplicateRegistration",
message = "The pipeline is already registered to the Reader. Registering pipeline multiple times is not allowed.");

Check warning on line 82 in opentelemetry-sdk/src/metrics/manual_reader.rs

View check run for this annotation

Codecov / codecov/patch

opentelemetry-sdk/src/metrics/manual_reader.rs#L80-L82

Added lines #L80 - L82 were not covered by tests
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Utkarsh's comment:
1.
name: "ManualReader.RegisterPipeline.DuplicateRegistration",
name: "ManualReader.DuplicateRegistration",
2.
replace error with `message

}
});
}
Expand Down
Loading