[proxy] github.com — ← back | site home | direct (HTTPS) ↗ | proxy home | ◑ dark◐ light

bpo-5001: More-informative multiprocessing error messages by drallensmith · Pull Request #3079 · python/cpython

drallensmith

added 7 commits

August 12, 2017 06:59
Replace assertions in error-reporting code with more-informative version that doesn't cause confusion over where and what the error is.

As suggested in PR comment by @pitrou, changing from SystemError; TypeError appears appropriate.
…by additional work)
…base

drallensmith changed the title bpo-5001: More-informative multiprocessing error messages (managers.py, queues.py) bpo-5001: More-informative multiprocessing error messages (managers.py, queues.py, pool.py)

Aug 17, 2017

drallensmith changed the title bpo-5001: More-informative multiprocessing error messages (managers.py, queues.py, pool.py) bpo-5001: More-informative multiprocessing error messages (managers.py, queues.py, pool.py, util.py)

Aug 18, 2017

drallensmith changed the title bpo-5001: More-informative multiprocessing error messages (managers.py, queues.py, pool.py, util.py) bpo-5001: More-informative multiprocessing error messages

Aug 19, 2017 

…once

OK, never mind.

…ke possible to merge

GadgetSteve pushed a commit to GadgetSteve/cpython that referenced this pull request

Sep 10, 2017 

* Make error message more informative

Replace assertions in error-reporting code with more-informative version that doesn't cause confusion over where and what the error is.

* Additional clarification + get travis to check

* Change from SystemError to TypeError

As suggested in PR comment by @pitrou, changing from SystemError; TypeError appears appropriate.

* NEWS file installation; ACKS addition (will do my best to justify it by additional work)

* Making current AssertionErrors in multiprocessing more informative

* Blurb added re multiprocessing managers.py, queues.py cleanup

* Further multiprocessing cleanup - went through pool.py

* Fix two asserts in multiprocessing/util.py

* Most asserts in multiprocessing more informative

* Didn't save right version

* Further work on multiprocessing error messages

* Correct typo

* Correct typo v2

* Blasted colon... serves me right for trying to work on two things at once

* Simplify NEWS entry

* Update 2017-08-18-17-16-38.bpo-5001.gwnthq.rst

* Update 2017-08-18-17-16-38.bpo-5001.gwnthq.rst

OK, never mind.

* Corrected (thanks to pitrou) error messages for notify

* Remove extraneous backslash in docstring.