diff options
author | Richard Oudkerk <shibturn@gmail.com> | 2013-10-28 23:11:58 (GMT) |
---|---|---|
committer | Richard Oudkerk <shibturn@gmail.com> | 2013-10-28 23:11:58 (GMT) |
commit | e90cedb71102c78b93839fd282f53c618defc023 (patch) | |
tree | 5fdf60f355610d7e441f97316947da1b9035e204 /Lib/multiprocessing | |
parent | 3797065ac55997741fd625a30a8308c04ee5c9b9 (diff) | |
download | cpython-e90cedb71102c78b93839fd282f53c618defc023.zip cpython-e90cedb71102c78b93839fd282f53c618defc023.tar.gz cpython-e90cedb71102c78b93839fd282f53c618defc023.tar.bz2 |
Issue #19425 -- a pickling error should not cause pool to hang.
Diffstat (limited to 'Lib/multiprocessing')
-rw-r--r-- | Lib/multiprocessing/pool.py | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/Lib/multiprocessing/pool.py b/Lib/multiprocessing/pool.py index fc9d904..0f2dab4 100644 --- a/Lib/multiprocessing/pool.py +++ b/Lib/multiprocessing/pool.py @@ -147,7 +147,8 @@ class Pool(object): self._task_handler = threading.Thread( target=Pool._handle_tasks, - args=(self._taskqueue, self._quick_put, self._outqueue, self._pool) + args=(self._taskqueue, self._quick_put, self._outqueue, + self._pool, self._cache) ) self._task_handler.daemon = True self._task_handler._state = RUN @@ -338,7 +339,7 @@ class Pool(object): debug('worker handler exiting') @staticmethod - def _handle_tasks(taskqueue, put, outqueue, pool): + def _handle_tasks(taskqueue, put, outqueue, pool, cache): thread = threading.current_thread() for taskseq, set_length in iter(taskqueue.get, None): @@ -349,9 +350,12 @@ class Pool(object): break try: put(task) - except IOError: - debug('could not put task on queue') - break + except Exception as e: + job, ind = task[:2] + try: + cache[job]._set(ind, (False, e)) + except KeyError: + pass else: if set_length: debug('doing set_length()') |