per this comment and our in-person discussion, I've renamed Sync to waitUntilEmpty() , thereby also making it private.  This is so that it's only used by Close() which is intended to be used when logging is dying down.