Just wanted a sanity check from someone more familiar with the parallel code.
I'm reviewing #15966 and the change is basically to clean up the child processes used for parallelization. As far as I could tell no one is calling this anywhere in the code base (until we are in #15920) so perhaps this defect was overlooked before, unless it is there for some reason (and thus not a defect). As far as I can tell all tests pass and the change cleans up the child processes. Thanks, Ben -- You received this message because you are subscribed to the Google Groups "sage-devel" group. To unsubscribe from this group and stop receiving emails from it, send an email to sage-devel+unsubscr...@googlegroups.com. To post to this group, send email to sage-devel@googlegroups.com. Visit this group at http://groups.google.com/group/sage-devel. For more options, visit https://groups.google.com/d/optout.
