Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/251#issuecomment-39004282
I don't understand what you are saying `SparkListenerShutDown` is a thing
you invented for this pull request, right? I'm not suggesting that just calling
`waitUntilEmpty` is identical to the `SparkListenerShutDown` approach. There
are some differences. In particular if an event is posted after `stop()` is
called it will just be ignored in my proposal. I'm fine with those semantics.
Could you describe a concrete problem with doing `waitUntilEmpty` inside of
`stop()`. And define it in terms of the current code and not the code added
here?
---
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.
---
