Message224369
Thank you for the patch. I've posted some review comments. Two further remarks:
- this makes the ProcessPool API slightly different from the ThreadPool one. I wonder if this is acceptable or not. Thoughts?
- the patch would need unit tests for the additional functionality |
|
Date |
User |
Action |
Args |
2014-07-31 01:42:56 | pitrou | set | recipients:
+ pitrou, bquinlan, tbrink, sbt, dan.oreilly |
2014-07-31 01:42:56 | pitrou | set | messageid: <1406770976.48.0.386585004514.issue11271@psf.upfronthosting.co.za> |
2014-07-31 01:42:56 | pitrou | link | issue11271 messages |
2014-07-31 01:42:56 | pitrou | create | |
|