Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20893
The change looks ok but the test being added is totally unrelated to it.
If you want to write a test, it should be not too hard to add something to
`SparkLauncherSuite` in `core/`. Start an in-process app that waits on a lock,
disconnect the handle, then wake up the lock so the app finishes successfully.
That should fail without your fix and work with it.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
