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, 
that SimpleQueue is not tested? I see a TestSimpleQueue test case in 
Lib/test/_test_multiprocessing.py.

haypo@selma$ ./python -m test test_multiprocessing_spawn -m TestSimpleQueue -R 
3:3
(...)
Tests result: SUCCESS


> 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 for longer than expected (e.g. in a 
> reference cycle, or with PyPy).

Oh ok, the garbage collector is able to close the file descriptors. The bug is 
when a SimpleQueue is not collected.

So again, I consider that a ResourceWarning is needed here. The purpose of a 
ResourceWarning is to warn when a leak may be kept alive longer than expected 
if it's not closed explicitly.

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue30966>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to