db-migrate/node-db-migrate

EventEmitter memory leaks

Open

#421 geöffnet am 4. Okt. 2016

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (2.296 Stars) (367 Forks)batch import
Refactoringgood first issuehelp wanted

Beschreibung

When using db-migrate programmatically for multiple databases, warnings are generated for EventEmitter leaks: Warning: Possible EventEmitter memory leak detected. 11 unhandledRejection listeners added.

Adding a unregisterEvents function to api.js should enable programmatic users to get around this. Such a function may also be used more generally to do proper cleanup.

Where registerEvent() is called: https://github.com/db-migrate/node-db-migrate/blob/master/api.js#L136-L147

Where registerEvents() is defined: https://github.com/db-migrate/node-db-migrate/blob/master/api.js#L136-L147

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Contributor Guide

Tech Stack
javascript
Domain
backenddatabase
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
mostly clear
Voraussetzungen
Basic Node.js knowledgeUnderstanding of EventEmitter
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60
Research-Richtung
First, study the existing `registerEvents` function in `api.js` (lines 136-147) to understand how event listeners are added. Then, design an `unregisterEvents` function that removes those listeners. Consider using `removeListener` or `removeAllListeners` for specific events. Finally, implement the function and export it from the module. No existing PR or comments provide a solution, so this is a clean feature addition.