Skip to content

Commit

Permalink
Update CHANGELOGs, clean up trace/span_processor
Browse files Browse the repository at this point in the history
  • Loading branch information
scottgerring committed Dec 5, 2024
1 parent 70edda1 commit d2a88ab
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 8 deletions.
3 changes: 3 additions & 0 deletions opentelemetry-otlp/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@

## vNext

- `OtlpHttpClient.shutdown` `TonicLogsClient.shutdown`, and `TonicTracesClient.shutdown` now explicitly return a result. The
semantics of the method have not changed, but you will have a new lint encouraging you to consume these results.

## 0.27.0

Released 2024-Nov-11
Expand Down
2 changes: 2 additions & 0 deletions opentelemetry-sdk/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@
transparent change.
[#2338](https://github.com/open-telemetry/opentelemetry-rust/pull/2338)
- `ResourceDetector.detect()` no longer supports timeout option.
- The trait functions `LogExporter.shutdown` and `TraceExporter.shutdown` now explicitly return a result. The
semantics of the method have not changed, but you will have a new lint encouraging you to consume these results.

## 0.27.1

Expand Down
9 changes: 1 addition & 8 deletions opentelemetry-sdk/src/trace/span_processor.rs
Original file line number Diff line number Diff line change
Expand Up @@ -149,14 +149,7 @@ impl SpanProcessor for SimpleSpanProcessor {
}

fn shutdown(&self) -> TraceResult<()> {
if let Ok(mut exporter) = self.exporter.lock() {
exporter.shutdown()?;
Ok(())
} else {
Err(TraceError::Other(
"SimpleSpanProcessor mutex poison at shutdown".into(),
))
}
self.exporter.lock()?.shutdown()
}

fn set_resource(&mut self, resource: &Resource) {
Expand Down
4 changes: 4 additions & 0 deletions opentelemetry-stdout/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,10 @@

## vNext

- `LogExporter.shutdown` and `SpanExporter.shutdown` now explicitly return a result. The
semantics of the method have not changed, but you will have a new lint encouraging you to consume these results.


## 0.27.0

Released 2024-Nov-11
Expand Down

0 comments on commit d2a88ab

Please sign in to comment.