db-migrate / db-migrate/node-db-migrate
EventEmitter memory leaks
- Ngôn ngữ chính
- JavaScript
- Star
- 2.3k
- Fork
- 361
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
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!](https://www.bountysource.com/issues/38131674-eventemitter-memory-leaks?utm_campaign=plugin&utm_content=tracker%2F73887&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F73887&utm_medium=issues&utm_source=github).
Hướng dẫn đóng góp
Hướng nghiên cứu
Bắt đầu trong api.js tại mã được liên kết của registerEvent() và registerEvents(), sau đó tái hiện việc sử dụng theo chương trình trên nhiều cơ sở dữ liệu để quan sát các cảnh báo của listener unhandledRejection. Định nghĩa việc dọn dẹp bằng unregisterEvents() tại đó và xác minh rằng việc sử dụng lặp lại không còn tích lũy các listener hoặc phát ra cảnh báo rò rỉ.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- javascript, node.js
- Lĩnh vực
- api, backend
- Loại issue
- Lỗi
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 45/100