Skip to content

Commit

Permalink
fix lint
Browse files Browse the repository at this point in the history
  • Loading branch information
lalitb committed Dec 24, 2024
1 parent 1b5f52b commit 76973f4
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions opentelemetry-sdk/src/logs/log_processor.rs
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,6 @@ enum BatchMessage {

/// A [`LogProcessor`] that buffers log records and reports
/// them at a pre-configured interval from a dedicated background thread.
// **Memory Management in BatchLogProcessor**
//
// The `BatchLogProcessor` manages memory through the following stages of log processing:
Expand All @@ -181,7 +180,6 @@ enum BatchMessage {
// to allocate them on the heap before entering the queue. Which means:
// - The entire `LogRecord` object, including its stack-allocated inline attributes, is moved to the heap.
// - Any overflow attributes already on the heap remain unaffected.

//
// 2. **Queue Management**:
// - Uses a **bounded synchronous channel** (`sync_channel`) with a maximum size defined by `max_queue_size`.
Expand All @@ -202,7 +200,7 @@ enum BatchMessage {
// - Ownership of the boxed records is transferred to the new vector, ensuring efficient
// memory usage without additional cloning.
// - The temporary vector is then used to construct references passed to the exporter via `LogBatch`.
// - The `export()` method receives references to the log records and `InstrumentationScope`
// - The exporter's `export()` method receives references to the log records and `InstrumentationScope`
// contained in `LogBatch`.
// - If the exporter requires retaining the log records (e.g., for retries or asynchronous
// operations), it must **clone** the records inside the `export()` implementation.
Expand Down

0 comments on commit 76973f4

Please sign in to comment.