Prossible perf_hooks memory leak detected

It seems nanobus creates a performance entry on event emit and never clear the entry, so after a long time of using, more and more entries of performance will be in memory.

key code here:

var emitTiming = nanotiming(this._name + "('" + eventName + "')")
  var listeners = this._listeners[eventName]
  if (listeners && listeners.length > 0) {
    this._emit(this._listeners[eventName], data)
  }

  if (this._starListeners.length > 0) {
    this._emit(this._starListeners, eventName, data, emitTiming.uuid)
  }
  emitTiming()

and warning message is:

(node:84217) Warning: Possible perf_hooks memory leak detected. There are 996 entries in the Performance Timeline. Use the clear methods to remove entries that are no longer needed or set performance.maxEntries equal to a higher value (currently the maxEntries is 150).

and my test code is here:

const nanobus = require('nanobus')
const bus = nanobus()

bus.on('foo', () => {})

for (let i=0; i<1000; i++) {
  bus.emit('foo', 'bus')
}

I see this message when I use webpack-serve which use nanobus as bus to emit events.

So I guess it’s a bug?

[yoshuawuyts]

Oh yeah, thought of another tweak we could do: we could detect the node version and switch off perf timings if the version is <10. Makes it so there’s less compat code to worry about 😄

[yoshuawuyts]

@Jokcy yep, that sounds about right. I was hoping the new timing API would have been backported to 9.x.x but unfortunately it hasn’t been. On the other hand: 9.x is only being supported until the end of the month so maybe it’s okay to not backport feature detection / support?