add comment

Author: lalitb

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

@@ -80,16 +80,16 @@ impl LogExporter for TonicLogsClient {
 
             let resource_logs = group_logs_by_resource_and_scope(batch, &self.resource);
 
-        otel_debug!(name: "TonicsLogsClient.CallingExport");
+            otel_debug!(name: "TonicsLogsClient.CallingExport");
 
-        client
-            .export(Request::from_parts(
-                metadata,
-                extensions,
-                ExportLogsServiceRequest { resource_logs },
-            ))
-            .await
-            .map_err(crate::Error::from)?;
+            client
+                .export(Request::from_parts(
+                    metadata,
+                    extensions,
+                    ExportLogsServiceRequest { resource_logs },
+                ))
+                .await
+                .map_err(crate::Error::from)?;
             Ok(())
         }
     }

opentelemetry-sdk/src/export/logs/mod.rs

@@ -79,6 +79,9 @@ pub trait LogExporter: Send + Sync + Debug {
     /// A `LogResult<()>`, which is a result type indicating either a successful export (with
     /// `Ok(())`) or an error (`Err(LogError)`) if the export operation failed.
     ///
+    /// Note:
+    /// The `Send` bound ensures the future can be safely moved across threads, which is crucial for multi-threaded async runtimes like Tokio.
+    /// Explicit lifetimes (`'a`) synchronize the lifetimes of `self`, `batch`, and the returned future.
     fn export<'a>(
         &'a self,
         batch: &'a LogBatch<'a>,