db-migrate/node-db-migrate

EventEmitter memory leaks

Open

#421 aberto em 4 de out. de 2016

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)JavaScript (367 forks)batch import
Refactoringgood first issuehelp wanted

Métricas do repositório

Stars
 (2.296 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Veja a função registerEvents em api.js, entenda como os eventos são adicionados e implemente uma função unregisterEvents para remover os listeners para limpeza.
Pilha de tecnologia
javascriptnodejs
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitNode.js
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75