lukechilds/node - node - Gitea: Git with a cup of tea

12923 Commits

131 Branches

446 Tags

325 MiB

Tree: b18fd41649

Commit Graph

Author	SHA1	Message	Date
Rich Trott	efbb654e0f	test: fix flaky cluster test on Windows 10 test-cluster-shared-leak was flaky on Windows 10. Remove unnecessary .send() calls and replace with .disconnect() to avoid spurious EPIPE. Fixes: https://github.com/nodejs/node/issues/4887 PR-URL: https://github.com/nodejs/node/pull/4934 Reviewed-By: James M Snell <jasnell@gmail.com>	9 years ago
Rich Trott	ec24d3767b	test: fix flaky test-cluster-shared-leak Wait for worker2 to come online before doing anything that might result in an EPIPE. Fixes flakiness of test on Windows. Fixes: https://github.com/nodejs/node/issues/3956 PR-URL: https://github.com/nodejs/node/pull/4510 Reviewed-By: Johan Bergström <bugs@bergstroem.nu> Reviewed-By: James M Snell<jasnell@gmail.com>	9 years ago
Rich Trott	2109708186	lib: fix cluster handle leak It is possible to cause a resource leak in SharedHandle. This commit fixes the leak. Fixes: https://github.com/nodejs/node/issues/2510 PR-URL: https://github.com/nodejs/node/pull/3510 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>	9 years ago

Author

SHA1

Message

Date

Rich Trott

efbb654e0f

test: fix flaky cluster test on Windows 10

test-cluster-shared-leak was flaky on Windows 10. Remove unnecessary
.send() calls and replace with .disconnect() to avoid spurious EPIPE.

Fixes: https://github.com/nodejs/node/issues/4887
PR-URL: https://github.com/nodejs/node/pull/4934
Reviewed-By: James M Snell <jasnell@gmail.com>

9 years ago

Rich Trott

ec24d3767b

test: fix flaky test-cluster-shared-leak

Wait for worker2 to come online before doing anything that might result
in an EPIPE. Fixes flakiness of test on Windows.

Fixes: https://github.com/nodejs/node/issues/3956
PR-URL: https://github.com/nodejs/node/pull/4510
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: James M Snell<jasnell@gmail.com>

9 years ago

Rich Trott

2109708186

lib: fix cluster handle leak

It is possible to cause a resource leak in SharedHandle. This commit
fixes the leak.

Fixes: https://github.com/nodejs/node/issues/2510
PR-URL: https://github.com/nodejs/node/pull/3510
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

9 years ago

3 Commits (b18fd416493cc084fe4cf65b2b2261435cc8d7fe)