fix(otlp-exporter-base): decrease default concurrency limit to 30 (#4211

) * fix(otlp-exporter-base): decrease concurrency limit to 30 * fix(changelog): add changelog entry
open-telemetry · Dec 13, 2023 · af4f7a9 · af4f7a9
1 parent dddab06
commit af4f7a9
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);