Code changes

Author: utpilla

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

@@ -23,12 +23,12 @@ pub struct LogBatch<'a> {
 
 /// The `LogBatchData` enum represents the data field of a `LogBatch`.
 /// It can either be:
-/// - A mutable reference to a vector of tuples, where each tuple consists of a `LogRecord` and an `InstrumentationScope`.
+/// - A shared reference to a vector of tuples, where each tuple consists of a `LogRecord` and an `InstrumentationScope`.
 /// - Or it can be a slice of tuples, where each tuple consists of a reference to a `LogRecord` and a reference to an `InstrumentationScope`.
 #[derive(Debug)]
 #[allow(clippy::vec_box)] // TODO: Revisit this. Clippy complains about using Box in a Vec, but it's done here for performant moves of the data between channel and the vec.
 enum LogBatchData<'a> {
-    BorrowedVec(&'a mut Vec<Box<(LogRecord, InstrumentationScope)>>), // Used by BatchProcessor which clones the LogRecords for its own use.
+    BorrowedVec(&'a Vec<Box<(LogRecord, InstrumentationScope)>>), // Used by BatchProcessor which clones the LogRecords for its own use.
     BorrowedSlice(&'a [(&'a LogRecord, &'a InstrumentationScope)]),
 }
 
@@ -53,9 +53,9 @@ impl<'a> LogBatch<'a> {
         }
     }
 
-    #[allow(clippy::vec_box)] // TODO: Revisit this. Clippy complains about using Box in a Vec, but it's done here for performant moves of the data between channel and the vec.
+    #[allow(clippy::vec_box)] // Clippy complains about using Box in a Vec, but it's done here for performant moves of the data between channel and the vec.
     pub(crate) fn new_with_owned_data(
-        data: &'a mut Vec<Box<(LogRecord, InstrumentationScope)>>,
+        data: &'a Vec<Box<(LogRecord, InstrumentationScope)>>,
     ) -> LogBatch<'a> {
         LogBatch {
             data: LogBatchData::BorrowedVec(data),