db-migrate/node-db-migrate

EventEmitter memory leaks

Open

#421 aperta il 4 ott 2016

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)JavaScript (367 fork)batch import
Refactoringgood first issuehelp wanted

Metriche repository

Star
 (2296 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Guarda la funzione registerEvents in api.js, capisci come vengono aggiunti gli eventi e implementa una funzione unregisterEvents per rimuovere i listener per la pulizia.
Tech stack
javascriptnodejs
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitNode.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75