classification
Title: LEaking files in concurrent.futures.process
Type: behavior Stage: resolved
Components: Windows, XML Versions: Python 3.8, Python 3.7
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: Nosy List: asvetlov, paul.moore, pitrou, steve.dower, tim.golden, tomMoral, zach.ware
Priority: normal Keywords: patch

Created on 2018-03-12 15:09 by tomMoral, last changed 2018-03-13 14:49 by steve.dower. This issue is now closed.

Pull Requests
URL Status Linked Edit
PR 6084 merged tomMoral, 2018-03-12 15:09
PR 6092 merged miss-islington, 2018-03-12 22:11
Messages (6)
msg313656 - (view) Author: Thomas Moreau (tomMoral) * Date: 2018-03-12 15:09
The recent changes introduced by https://github.com/python/cpython/pull/3895 leaks some file descriptors (the Pipe open in _ThreadWakeup).
They should be properly closed at shutdown.
msg313669 - (view) Author: Andrew Svetlov (asvetlov) * (Python committer) Date: 2018-03-12 17:18
New changeset 095ee415cee41bf24c3a1108c23307e5baf168dd by Andrew Svetlov (Thomas Moreau) in branch 'master':
bpo-33056 FIX leaking fd in concurrent.futures.ProcessPoolExecutor (#6084)
https://github.com/python/cpython/commit/095ee415cee41bf24c3a1108c23307e5baf168dd
msg313733 - (view) Author: Antoine Pitrou (pitrou) * (Python committer) Date: 2018-03-13 09:11
New changeset f216cbf9ab704da98146a25d57ff0e85aecb49da by Antoine Pitrou (Miss Islington (bot)) in branch '3.7':
bpo-33056 FIX leaking fd in concurrent.futures.ProcessPoolExecutor (GH-6084) (#6092)
https://github.com/python/cpython/commit/f216cbf9ab704da98146a25d57ff0e85aecb49da
msg313734 - (view) Author: Antoine Pitrou (pitrou) * (Python committer) Date: 2018-03-13 09:12
Andrew: where there are multiprocessing issues, could you please nosy me so that I get a chance to review?

Thomas: thanks for spotting this and thanks for the fix!
msg313738 - (view) Author: Andrew Svetlov (asvetlov) * (Python committer) Date: 2018-03-13 10:16
Antoine sorry, will do next time
msg313756 - (view) Author: Steve Dower (steve.dower) * (Python committer) Date: 2018-03-13 14:49
Andrew - you can search for some module names in the nosy list to add the designated people for them. The components list doesn't include all of them (perhaps it should?).
History
Date User Action Args
2018-03-13 14:49:02steve.dowersetmessages: + msg313756
2018-03-13 10:16:18asvetlovsetnosy: + paul.moore, tim.golden, zach.ware, steve.dower
messages: + msg313738
components: + Windows, XML, - Library (Lib)
2018-03-13 09:12:21pitrousetstatus: open -> closed
versions: + Python 3.8
messages: + msg313734

resolution: fixed
stage: patch review -> resolved
2018-03-13 09:11:00pitrousetnosy: + pitrou
messages: + msg313733
2018-03-12 22:11:47miss-islingtonsetkeywords: + patch
stage: patch review
pull_requests: + pull_request5854
2018-03-12 17:18:46asvetlovsetnosy: + asvetlov
messages: + msg313669
2018-03-12 15:09:15tomMoralcreate