Github user kanzhang commented on the pull request:

    https://github.com/apache/spark/pull/366#issuecomment-40031940
  
    Thanks for getting rid of the 100ms wait. I noticed it but didn't have 
better idea.
    
    I think this patch addresses SPARK-1407. Without this patch, even if 
sc.stop() is called, it will only close the logger and flush whatever is 
already in its write buffer. If, at this point, some events are yet to be 
drained from event queue, when listenerThread tries to post them to logger 
later on, the same Filesystem is closed exception will be encountered. 
Consequently event log file will be incomplete (same as in SPARK-1407). 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to