[issue30966] multiprocessing.queues.SimpleQueue leaks 2 fds

2020-04-27 Thread STINNER Victor
STINNER Victor added the comment: bpo-23267 is marked as a duplicate of this issue. -- ___ Python tracker ___ ___ Python-bugs-list

[issue30966] multiprocessing.queues.SimpleQueue leaks 2 fds

2020-04-27 Thread STINNER Victor
Change by STINNER Victor : -- keywords: +patch pull_requests: +19057 stage: -> patch review pull_request: https://github.com/python/cpython/pull/19735 ___ Python tracker ___ _

[issue30966] multiprocessing.queues.SimpleQueue leaks 2 fds

2017-07-20 Thread STINNER Victor
Changes by STINNER Victor : -- pull_requests: +2830 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mai

[issue30966] multiprocessing.queues.SimpleQueue leaks 2 fds

2017-07-19 Thread STINNER Victor
STINNER Victor added the comment: > Anyone who thinks those objects stay alive too long can submit a PR for > SimpleQueue and Pool to close them eagerly. I started with SimpleQueue for the master branch: https://github.com/python/cpython/pull/2760 -- __

[issue30966] multiprocessing.queues.SimpleQueue leaks 2 fds

2017-07-19 Thread STINNER Victor
Changes by STINNER Victor : -- pull_requests: +2819 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mai

[issue30966] multiprocessing.queues.SimpleQueue leaks 2 fds

2017-07-19 Thread Antoine Pitrou
Antoine Pitrou added the comment: Anyone who thinks those objects stay alive too long can submit a PR for SimpleQueue and Pool to close them eagerly. -- ___ Python tracker ___ _

[issue30966] multiprocessing.queues.SimpleQueue leaks 2 fds

2017-07-19 Thread STINNER Victor
Changes by STINNER Victor : -- versions: -Python 3.3, Python 3.4 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe

[issue30966] multiprocessing.queues.SimpleQueue leaks 2 fds

2017-07-19 Thread STINNER Victor
STINNER Victor added the comment: This issue is marked as also affecting Python 3.7. I don't see a SimpleQueue.close() method in master, but I don't remind any resource warning when running tests on master neither. Does it mean that our test runner miss such file descriptor leak? Or worse, th

[issue30966] multiprocessing.queues.SimpleQueue leaks 2 fds

2017-07-19 Thread STINNER Victor
STINNER Victor added the comment: See also my bpo-30171: "Emit ResourceWarning in multiprocessing Queue destructor". -- ___ Python tracker ___ __

[issue30966] multiprocessing.queues.SimpleQueue leaks 2 fds

2017-07-19 Thread STINNER Victor
Changes by STINNER Victor : -- nosy: +davin, pitrou ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mai

[issue30966] multiprocessing.queues.SimpleQueue leaks 2 fds

2017-07-19 Thread STINNER Victor
STINNER Victor added the comment: On Python 3.6+, regrtest is able to detect file descriptor leaks when using --huntrleaks. Is someone interested to backport this feature to 3.5 and/or 2.7 which would mean fix all FD leaks in our test suite? If someone wants to work on that, I would suggest to

[issue30966] multiprocessing.queues.SimpleQueue leaks 2 fds

2017-07-19 Thread STINNER Victor
Changes by STINNER Victor : -- nosy: +haypo ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python

[issue30966] multiprocessing.queues.SimpleQueue leaks 2 fds

2017-07-19 Thread Xiang Zhang
Xiang Zhang added the comment: Get this solved then we could also solve #23267, which reports pipes leak in pool due to using SimpleQueue. -- nosy: +xiang.zhang ___ Python tracker _

[issue30966] multiprocessing.queues.SimpleQueue leaks 2 fds

2017-07-19 Thread Armin Rigo
New submission from Armin Rigo: multiprocessing.queues.SimpleQueue should have a close() method. This is needed to explicitly release the two file descriptors of the Pipe used internally. Without it, the file descriptors leak if a reference to the SimpleQueue object happens to stay around fo