Message159321
mp_resource_sharer_stop.patch: you should add a timeout argument to stop() instead of hardcoding a timeout of 5 seconds. It is maybe safer to block until the thread exits by default (so timeout=None by default).
For the new method: it may be nice to document it. Having to import resource_sharer from multiprocessing.reduction is maybe not the best possible API :-/
+ from multiprocessing.reduction import resource_sharer
+ resource_sharer.stop()
> Also, applicative code could be affected.
What is the effect of the patch? For example, on CTRL+c? I don't know the multiprocessing module nor this "resource sharer" thread. |
|
Date |
User |
Action |
Args |
2012-04-25 17:02:00 | vstinner | set | recipients:
+ vstinner, pitrou, neologix, sbt |
2012-04-25 17:02:00 | vstinner | set | messageid: <1335373320.53.0.257986184198.issue14666@psf.upfronthosting.co.za> |
2012-04-25 17:01:59 | vstinner | link | issue14666 messages |
2012-04-25 17:01:59 | vstinner | create | |
|