Removed duplicate appending of log records (#504)

itsniper · web-flow · commit 0106a8628540 · 2024-01-25T17:55:31.000+01:00
diff --git a/Sources/Exporters/OpenTelemetryProtocolHttp/logs/OtlpHttpLogExporter.swift b/Sources/Exporters/OpenTelemetryProtocolHttp/logs/OtlpHttpLogExporter.swift
@@ -23,9 +23,7 @@ public class OtlpHttpLogExporter : OtlpHttpExporterBase, LogRecordExporter {
   }
   
   public func export(logRecords: [OpenTelemetrySdk.ReadableLogRecord], explicitTimeout: TimeInterval? = nil) -> OpenTelemetrySdk.ExportResult {
-    pendingLogRecords.append(contentsOf: logRecords)
     var sendingLogRecords: [ReadableLogRecord] = []
-    
     exporterLock.withLockVoid {
       pendingLogRecords.append(contentsOf: logRecords)
       sendingLogRecords = pendingLogRecords

Original file line number	Diff line number	Diff line change
`@@ -23,9 +23,7 @@ public class OtlpHttpLogExporter : OtlpHttpExporterBase, LogRecordExporter {`
`23`	`23`	`}`
`24`	`24`
`25`	`25`	`public func export(logRecords: [OpenTelemetrySdk.ReadableLogRecord], explicitTimeout: TimeInterval? = nil) -> OpenTelemetrySdk.ExportResult {`
`26`		`- pendingLogRecords.append(contentsOf: logRecords)`
`27`	`26`	`var sendingLogRecords: [ReadableLogRecord] = []`
`28`		`-`
`29`	`27`	`exporterLock.withLockVoid {`
`30`	`28`	`pendingLogRecords.append(contentsOf: logRecords)`
`31`	`29`	`sendingLogRecords = pendingLogRecords`