Github user squito commented on the pull request:

    https://github.com/apache/spark/pull/8559#issuecomment-219840630
  
    @mateiz @markhamstra @kayousterhout I've updated this to avoid changing the 
scheduler at all, and instead just run the mock backend in another thread.  It 
slightly complicates writing the tests, but avoids a (small) race in the tests 
and was probably necessary when we got to more complicated tests in any case.
    
    In particular I want this for more tests on blacklisting 
([SPARK-8426](https://github.com/apache/spark/pull/8760)), but I think there 
are a handful of open scheduler issues which will benefit from it.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to