MaxListenersExceededWarning: Possible EventEmitter memory leak detected.

[markNZed]

Describe the bug
Every time we create a Redis keyv an event is added to monitor Redis errors

keyv/packages/redis/src/index.ts

Line 39 in c38b524

this.redis.on('error', (error: Error) => this.emit('error', error));

If we add many keyv this is going to exceed the default max number of listeners (10).

The warning looks like:

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

I'm unsure if this is a bug - the default limit can be raised. I'm not sure of a good way to address this without adding a lot of complication. I thought I would mention it in case someone does see a solution.

[jaredwray]

What we have seen is that people just increase the max listeners when they are using multiple instances. Also, we will be looking to change this in v5 👉 #868