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: #31022

PR-URL: #36794
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
  • Loading branch information
Trott authored and targos committed May 1, 2021
1 parent a800b5b commit 682d0a9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion benchmark/http/headers.js
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 682d0a9

Please sign in to comment.