Merge pull request #172 from open-telemetry/main

malkia · web-flow · commit 4dcd7cb49424 · 2025-02-04T09:30:57.000-08:00
[SDK] Fix BatchLogRecordProcessor to instrument shutdown (open-telemetry#3262)
diff --git a/sdk/src/logs/batch_log_record_processor.cc b/sdk/src/logs/batch_log_record_processor.cc
@@ -218,7 +218,7 @@ void BatchLogRecordProcessor::DoBackgroundWork()
     if (synchronization_data_->is_shutdown.load() == true)
     {
       DrainQueue();
-      return;
+      break;
     }
 
     auto start = std::chrono::steady_clock::now();

Original file line number	Diff line number	Diff line change
`@@ -218,7 +218,7 @@ void BatchLogRecordProcessor::DoBackgroundWork()`
`218`	`218`	`if (synchronization_data_->is_shutdown.load() == true)`
`219`	`219`	`{`
`220`	`220`	`DrainQueue();`
`221`		`- return;`
	`221`	`+ break;`
`222`	`222`	`}`
`223`	`223`
`224`	`224`	`auto start = std::chrono::steady_clock::now();`