fastify/benchmarks

Use throughput in result instead of requests

hekike opened this issue · 1 comments

Hi,

I'm not sure that I miss something or is it a bug:

autocannon-compare determinates the winner based on the throughput.
See: https://github.com/mcollina/autocannon-compare/blob/9ba26c51775ba33986873087ccd69786fd635359/compare.js#L19

Shouldn't benchmarks use throughput for the diff calculation and in the result object instead of requests?
See:

} else if (comp.aWins) {