diff options
author | Robert Nagy <ronagy@icloud.com> | 2019-12-15 15:44:15 +0300 |
---|---|---|
committer | Rich Trott <rtrott@gmail.com> | 2019-12-18 04:36:31 +0300 |
commit | ea86f8c902e13d8cff0e01007ed0d978f59fe132 (patch) | |
tree | ca4db1087100c122d527a6b6b959052c4085f250 /benchmark | |
parent | ec4bf4ae9b7d9b155492b6e0c3c7e049708c3966 (diff) |
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: https://github.com/nodejs/node/pull/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>
Diffstat (limited to 'benchmark')
-rw-r--r-- | benchmark/streams/writable-manywrites.js | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/benchmark/streams/writable-manywrites.js b/benchmark/streams/writable-manywrites.js index 0ed38d0357a..ebd83779b76 100644 --- 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(); } |