[06:03:32] <Krinkle>	 tgr|away: destruct might be unreliable, that makes sense to me. That too may have improved. Shutdown I'm more comfortable reling on. We have a fairly extensive failure emulator in prod we can use to confirm this. We also rely on it for profiling and upstream seems to rely on it by default and hasn't changed in all those years. In any event, it's not worth removing if it's harmless. But it's perhaps worth confirming if it's causing harm. 
[06:03:32] <Krinkle>	 Ie does upstream actually support what we do, to close it "early" but presumably keep logging and flush again later.