[issue23051] multiprocessing.pool methods imap()[_unordered()] deadlock

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

[issue23051] multiprocessing.pool methods imap()[_unordered()] deadlock

Serhiy Storchaka added the comment: Thank you for your contribution Alon and Davin. -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue23051] multiprocessing.pool methods imap()[_unordered()] deadlock

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 #23

[issue23051] multiprocessing.pool methods imap()[_unordered()] deadlock

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

[issue23051] multiprocessing.pool methods imap()[_unordered()] deadlock

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

[issue23051] multiprocessing.pool methods imap()[_unordered()] deadlock

Changes by Serhiy Storchaka : -- assignee: -> serhiy.storchaka ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue23051] multiprocessing.pool methods imap()[_unordered()] deadlock

Changes by Davin Potts : Added file: http://bugs.python.org/file38363/issue_23051_revised_fix_and_tests_v27.patch ___ Python tracker ___ ___

[issue23051] multiprocessing.pool methods imap()[_unordered()] deadlock

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: http://

[issue23051] multiprocessing.pool methods imap()[_unordered()] deadlock

Serhiy Storchaka added the comment: Added comments on Rietveld. -- nosy: +serhiy.storchaka stage: commit review -> patch review ___ Python tracker ___ ___

[issue23051] multiprocessing.pool methods imap()[_unordered()] deadlock

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 ___

[issue23051] multiprocessing.pool methods imap()[_unordered()] deadlock

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 ___

[issue23051] multiprocessing.pool methods imap()[_unordered()] deadlock

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 advertis

[issue23051] multiprocessing.pool methods imap()[_unordered()] deadlock

Changes by Davin Potts : -- nosy: +davin ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.or

[issue23051] multiprocessing.pool methods imap()[_unordered()] deadlock

Changes by Josh Rosenberg : -- nosy: +josh.r ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pytho

[issue23051] multiprocessing.pool methods imap()[_unordered()] deadlock

Changes by Terry J. Reedy : -- title: multiprocessing.pool methods imap() and imap_unordered() cause deadlock -> multiprocessing.pool methods imap()[_unordered()] deadlock ___ Python tracker __