[issue31308] forkserver process isn't re-launched if it died

2017-11-03 Thread Antoine Pitrou
Change by Antoine Pitrou : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ ___

[issue31308] forkserver process isn't re-launched if it died

2017-11-03 Thread Antoine Pitrou
Antoine Pitrou added the comment: New changeset 019c99f325287741d1e0eefeef2b75c8e00b884f by Antoine Pitrou in branch '3.6': [3.6] bpo-31308: If multiprocessing's forkserver dies, launch it again when necessary (GH-3246) (#4252) https://github.com/python/cpython/commit/019c99f325287741d1e0eefe

[issue31308] forkserver process isn't re-launched if it died

2017-11-03 Thread Antoine Pitrou
Change by Antoine Pitrou : -- keywords: +patch pull_requests: +4214 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscr

[issue31308] forkserver process isn't re-launched if it died

2017-11-03 Thread Antoine Pitrou
Antoine Pitrou added the comment: New changeset fc6b348b12ad401cab0261b7b71a65c60a08c0a8 by Antoine Pitrou in branch 'master': bpo-31308: If multiprocessing's forkserver dies, launch it again when necessary (#3246) https://github.com/python/cpython/commit/fc6b348b12ad401cab0261b7b71a65c60a08c

[issue31308] forkserver process isn't re-launched if it died

2017-11-03 Thread Antoine Pitrou
Antoine Pitrou added the comment: Davin, I think I'm going to merge the PR for this. If you object it, it can still be reverted later. -- ___ Python tracker ___ _

[issue31308] forkserver process isn't re-launched if it died

2017-10-16 Thread Matthew Rocklin
Matthew Rocklin added the comment: >From a user's perspective I would definitely appreciate forkserver being >protected from SIGINT. This bug affects me in practice. If the forkserver >goes down I personally have no objection to it restarting automatically, >though I appreciate that I have

[issue31308] forkserver process isn't re-launched if it died

2017-10-01 Thread Antoine Pitrou
Antoine Pitrou added the comment: Looks like this will not make into 3.6.3. I'd like to sum up the discussion a bit: - I think at the very least least we must protect the forkserver against SIGINT, like the semaphore tracker already is - I think it's beneficial to also restart it on demand if

[issue31308] forkserver process isn't re-launched if it died

2017-09-11 Thread Antoine Pitrou
Antoine Pitrou added the comment: > 1) The implicit restart of the forkserver process seems in conflict with the > zen of making things explicit. Well, the forkserver process really is an implementation detail. It already is started on-demand, so restarting it if needed sounded like a natural

[issue31308] forkserver process isn't re-launched if it died

2017-09-10 Thread Davin Potts
Davin Potts added the comment: I have two concerns with this: 1) The implicit restart of the forkserver process seems in conflict with the zen of making things explicit. 2) This would seem to make forkserver's behavior inconsistent with the behavior of things like the Manager which similarly cr

[issue31308] forkserver process isn't re-launched if it died

2017-08-30 Thread Antoine Pitrou
Changes by Antoine Pitrou : -- stage: needs patch -> patch review ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe

[issue31308] forkserver process isn't re-launched if it died

2017-08-30 Thread Antoine Pitrou
Changes by Antoine Pitrou : -- pull_requests: +3291 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mai

[issue31308] forkserver process isn't re-launched if it died

2017-08-30 Thread Antoine Pitrou
Changes by Antoine Pitrou : -- nosy: +davin ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python

[issue31308] forkserver process isn't re-launched if it died

2017-08-30 Thread Antoine Pitrou
New submission from Antoine Pitrou: It may happen that the forkserver process dies (for example if SIGINT is received because the user pressed Ctrl-C) while the parent process is still alive. In that case, if the parent tries to create a new Process instance, an exception is received. The ex