Reported throughput very low about benchmarks

[love-lena]

Well we can see from the logs that it is express-route-prefix. But even when running just one test by itself, I get numbers in the 10s. I think there are two different errors going on, but I am continuing to investigate.

[RafaelGSS]

Now it looks fixed in the repository. The website needs to be updated again. @luisorbaiceta could you help us with it?

[luisorbaiceta]

Done!

[love-lena]

If this is a "good first issue" I could have a crack at it. I've just picked up fastify recently so no promises. Anything I need to know about this package?

[Eomm]

The pipeline shows a lot of errors: https://github.com/fastify/benchmarks/runs/5476672680?check_suite_focus=true

We should understand what has changed in the GH pipelines that have triggered this error. Maybe the default node.js version has been updated?

[love-lena]

It actually looks like there were huge amounts of errors even before the numbers tanked.
Like 160,000 lines of

(node:2441) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 close listeners added to [Socket]. Use emitter.setMaxListeners() to increase limit
(node:2441) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 error listeners added to [Socket]. Use emitter.setMaxListeners() to increase limit

Are these known/okay?

[Eomm]

Are these known/okay?

I don't think so 😞

cc @fastify/benchmarks

[RafaelGSS]

@love-lena I would run each of those benchmarks individually until finding the one who's showing those warnings

[luisorbaiceta]

Now it looks fixed in the repository. The website needs to be updated again. @luisorbaiceta could you help us with it?

Sure! Would a redeploy be enough?

[RafaelGSS]

I think so

[luisorbaiceta]

Yep

https://deploy-preview-318--fastify-website.netlify.app/

[RafaelGSS]

It's fixed. Thank you!