fix(otlp-exporter-base): decrease default concurrency limit to 30 (op…

…en-telemetry#4211) * fix(otlp-exporter-base): decrease concurrency limit to 30 * fix(changelog): add changelog entry
Zirak · Sep 14, 2024 · 08fca88 · 08fca88
1 parent 6d944d6
commit 08fca88
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -9,6 +9,10 @@ For experimental package changes, see the [experimental CHANGELOG](experimental/
 
 ### :boom: Breaking Change
 
+* fix(otlp-exporter-base)!: decrease default concurrency limit to 30 [#4211](https://github.com/open-telemetry/opentelemetry-js/pull/4211) @pichlermarc
+  * fixes a memory leak on prolonged collector unavailability
+  * this change is marked as breaking as it changes defaults
+
 ### :rocket: (Enhancement)
 
 ### :books: (Refine Doc)

diff --git a/experimental/packages/otlp-exporter-base/src/OTLPExporterBase.ts b/experimental/packages/otlp-exporter-base/src/OTLPExporterBase.ts
@@ -57,7 +57,7 @@ export abstract class OTLPExporterBase<
     this._concurrencyLimit =
       typeof config.concurrencyLimit === 'number'
         ? config.concurrencyLimit
-        : Infinity;
+        : 30;
 
     this.timeoutMillis = configureExporterTimeout(config.timeoutMillis);