Skip to content

Commit 99e1109

Browse files
committed
Merge branch 'log-processor-optimize' of github.com:lalitb/opentelemetry-rust into log-processor-optimize
2 parents af1b590 + dd14edd commit 99e1109

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

opentelemetry-otlp/src/exporter/http/logs.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ impl LogExporter for OtlpHttpClient {
1919
_ => Err(LogError::Other("exporter is already shut down".into())),
2020
})?;
2121

22-
//TBD :avoid cloning and work only on borrowed logdata
22+
//TODO :avoid cloning when logdata is borrowed?
2323
let owned_batch = batch
2424
.into_iter()
2525
.map(|cow_log_data| cow_log_data.into_owned()) // Converts Cow to owned LogData

opentelemetry-otlp/src/exporter/tonic/logs.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ impl LogExporter for TonicLogsClient {
6666
None => return Err(LogError::Other("exporter is already shut down".into())),
6767
};
6868

69-
// TBD: We can avoid cloning and work only on logdata reference
69+
// TODO: Avoid cloning when logdata is borrowed?
7070
let resource_logs = {
7171
batch
7272
.into_iter()

0 commit comments

Comments
 (0)