From cd8a64f14ca312dcce2fd8e1343803e29ab0c069 Mon Sep 17 00:00:00 2001 From: cjihrig Date: Sat, 30 Mar 2024 23:12:51 -0400 Subject: [PATCH] test_runner: disable highWatermark on TestsStream The default highWatermark of 16 on the TestsStream class can have a substantial impact on reporting performance. This commit sets the TestsStream highWatermark to a very large value and lets the destination streams (which are more likely to have meaningful highWatermarks) handle backpressure. --- lib/internal/test_runner/tests_stream.js | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/lib/internal/test_runner/tests_stream.js b/lib/internal/test_runner/tests_stream.js index 1b86aeeaff2800..83b2a903218df1 100644 --- a/lib/internal/test_runner/tests_stream.js +++ b/lib/internal/test_runner/tests_stream.js @@ -2,6 +2,7 @@ const { ArrayPrototypePush, ArrayPrototypeShift, + NumberMAX_SAFE_INTEGER, Symbol, } = primordials; const Readable = require('internal/streams/readable'); @@ -12,7 +13,11 @@ class TestsStream extends Readable { #canPush; constructor() { - super({ __proto__: null, objectMode: true }); + super({ + __proto__: null, + objectMode: true, + highWaterMark: NumberMAX_SAFE_INTEGER, + }); this.#buffer = []; this.#canPush = true; }