Commit 1333798
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 <ruben@bridgewater.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>1 parent e6907f0 commit 1333798
1 file changed
Lines changed: 8 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -20,8 +20,13 @@ function main({ n, sync }) { | |||
| 20 | 20 | }; | |
| 21 | 21 | ||
| 22 | 22 | bench.start(); | |
| 23 | - for (var k = 0; k < n; ++k) { | ||
| 24 | - s.write(b); | ||
| 23 | + | ||
| 24 | + let k = 0; | ||
| 25 | + function run() { | ||
| 26 | + while (k++ < n && s.write(b)); | ||
| 27 | + if (k >= n) | ||
| 28 | + bench.end(n); | ||
| 25 | 29 | } | |
| 26 | - bench.end(n); | ||
| 30 | + s.on('drain', run); | ||
| 31 | + run(); | ||
| 27 | 32 | } | |
0 commit comments