Skip to content

Commit

Permalink
benchmark: fix http/headers.js with test-double
Browse files Browse the repository at this point in the history
The http/headers.js benchmark fails if wrk or autocannon are not
installed. This is because the benchmark exceeds the default permissible
length for HTTP headers. Reduce the largest benchmarks to fit within the
8192 default as that is what we should be optimizing for.

Fixes: nodejs#31022

PR-URL: nodejs#36794
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Juan José Arboleda <[email protected]>
Reviewed-By: Anto Aravinth <[email protected]>
  • Loading branch information
Trott committed Jan 7, 2021
1 parent 9ac4047 commit 1e35d58
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion benchmark/http/headers.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ const common = require('../common.js');
const http = require('http');

const bench = common.createBenchmark(main, {
n: [10, 1000],
n: [10, 600],
len: [1, 100],
duration: 5
});
Expand Down

0 comments on commit 1e35d58

Please sign in to comment.