benchmark: remove problematic tls params · nodejs/node@31ec443 · GitHub
Skip to content

Commit 31ec443

Browse files
mscdexMylesBorins
authored andcommitted
benchmark: remove problematic tls params
These very small values can cause crashes/exceptions to occur on some systems because most time is spent in V8 GC or in parts of node core that are not being tested (e.g. streams). PR-URL: #31816 Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent c0ba6ec commit 31ec443

3 files changed

Lines changed: 4 additions & 4 deletions

File tree

benchmark/tls/secure-pair.js

Lines changed: 1 addition & 1 deletion

benchmark/tls/throughput.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ const common = require('../common.js');
33
const bench = common.createBenchmark(main, {
44
dur: [5],
55
type: ['buf', 'asc', 'utf'],
6-
size: [2, 1024, 1024 * 1024, 4 * 1024 * 1024, 16 * 1024 * 1024]
6+
size: [100, 1024, 1024 * 1024, 4 * 1024 * 1024, 16 * 1024 * 1024]
77
});
88

99
const fixtures = require('../../test/common/fixtures');

test/benchmark/test-benchmark-tls.js

Lines changed: 2 additions & 2 deletions

0 commit comments

Comments
 (0)