Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/1255#issuecomment-47435516
@andrewor14 this looks good, but one thing, could we write some basic unit
tests to cover the behavior of this listener (at least to test the specific
case here). One of the major benefits of going through this event based model
is that it should be pretty easy to write tests.
---
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.
---
