Gregory Szorc added the comment:
For posterity, I think we ran into a similar problem in
https://bugzilla.mozilla.org/show_bug.cgi?id=1191877, where our code uses
apply_async():
11:09:47 INFO - Exception in thread Thread-2:
11:09:47 INFO - Traceback (most recent call last):
11:09:47
Serhiy Storchaka added the comment:
Thank you for your contribution Alon and Davin.
--
resolution: - fixed
stage: patch review - resolved
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23051
Roundup Robot added the comment:
New changeset 525ccfcc55f7 by Serhiy Storchaka in branch '3.4':
Issue #23051: multiprocessing.Pool methods imap() and imap_unordered() now
https://hg.python.org/cpython/rev/525ccfcc55f7
New changeset 7891d084a9ad by Serhiy Storchaka in branch 'default':
Issue
Davin Potts added the comment:
After pondering it for two days and coming back to it with hopefully fresh
eyes, I believe that changing the for-loop to a while-loop is not overall
easier to understand -- I would lean towards keeping the for-loop.
I do think the change to the while-loop very
Changes by Serhiy Storchaka storch...@gmail.com:
--
assignee: - serhiy.storchaka
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23051
___
___
Serhiy Storchaka added the comment:
May be the code would cleaner when convert the for loop to the while loop
and wrap in try/except only next()?
--
Added file: http://bugs.python.org/file38377/issue_23051_4-3.4.patch
___
Python tracker
Serhiy Storchaka added the comment:
Added comments on Rietveld.
--
nosy: +serhiy.storchaka
stage: commit review - patch review
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23051
___
Changes by Davin Potts pyt...@discontinuity.net:
Added file:
http://bugs.python.org/file38363/issue_23051_revised_fix_and_tests_v27.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23051
___
Davin Potts added the comment:
Updated (1) the patch for default/3.5 and 3.4 and (2) the patch for 2.7 to
reflect recommendations from the review.
Thanks goes to Serhiy for the helpful review and especially the suggestion on
better future-proofing in the tests.
--
Added file:
Davin Potts added the comment:
For my part, I'm now good with all aspects of the patch supplied by Alon.
I have a working set of tests that will be attached in the next day or two
after seeing them behave on more than one platform.
--
stage: - test needed
Antoine Pitrou added the comment:
The patch would at least need to add a unit test in order to avoid regressions.
--
nosy: +pitrou
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23051
___
Changes by Davin Potts pyt...@discontinuity.net:
--
nosy: +davin
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23051
___
___
Python-bugs-list
Davin Potts added the comment:
Successfully reproduced the behavior playing through variations with the
supplied examples (very helpful) on OS X v10.9.5 both in 2.7.9 and the default
branch -- adding version 3.5 to issue.
Also successfully verified that the supplied patches do just as
Changes by Terry J. Reedy tjre...@udel.edu:
--
title: multiprocessing.pool methods imap() and imap_unordered() cause deadlock
- multiprocessing.pool methods imap()[_unordered()] deadlock
___
Python tracker rep...@bugs.python.org
Changes by Josh Rosenberg shadowranger+pyt...@gmail.com:
--
nosy: +josh.r
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23051
___
___
15 matches
Mail list logo