Davin Potts added the comment:

This is a nice example demonstrating what I agree is a problem with the current 
implementation of close.

A practical concern with what I believe is being proposed in your trivial fix:  
if the workers are engaged in very long-running tasks (and perhaps slowly 
writing their overly large results to the results queue) then we would have to 
wait for quite a long time for these other workers to reach their natural 
completion.

That said, I believe close should in fact behave just that way and have us 
subsequently wait for the others to be completed.  It is not close's job to 
attempt to address the general concern I bring up.

This change could be felt by people who have written their code to expect the 
result handler's immediate shutdown if there are no other visible results -- it 
is difficult to imagine what the impact would be.

This is my long-winded way of saying it seems very sensible and welcome to me 
if you took the time to prepare a patch.

----------
stage:  -> needs patch

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue23992>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to