benchmark: update manywrites back pressure

Make manywrites benchmark a bit more realistic by taking back pressure into account. Otherwise memory usage would no correspond well with real world usage. PR-URL: #30977 Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Jeremiah Senkpiel <[email protected]> Reviewed-By: Rich Trott <[email protected]>
nodejs · Jan 3, 2020 · 0e864a3 · 0e864a3
1 parent d36ae62
commit 0e864a3
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/benchmark/streams/writable-manywrites.js b/benchmark/streams/writable-manywrites.js
@@ -20,8 +20,13 @@ function main({ n, sync }) {
   };
 
   bench.start();
-  for (var k = 0; k < n; ++k) {
-    s.write(b);
+
+  let k = 0;
+  function run() {
+    while (k++ < n && s.write(b));
+    if (k >= n)
+      bench.end(n);
   }
-  bench.end(n);
+  s.on('drain', run);
+  run();
 }