Github user srowen commented on the issue:

    https://github.com/apache/spark/pull/15706
  
    I think it may only involve the same exception but I am not sure it is the 
same. 
    
    I don't like removing tests of course but I think I conclude that the issue 
you intend to fix is fixed. It is the test that has a problem. We do want to 
make the change but need a test that will pass. If there is no obvious 
workaround for what looks to me like a fairly obscure class loader issue, then 
we could make this test not use the REPL.
    
    It would still test something useful and catch some potential regressions. 
It would not evidently catch this one though I suppose it is unlikely this 
exact issue would manifest again.  
    
    I could live with a simpler test to make this work and get the change into 
2.0.2 because otherwise it has to be reverted for now. 


---
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.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to