GitHub user jiangxb1987 opened a pull request:
https://github.com/apache/spark/pull/21908
[MINOR][CORE][TEST] Fix afterEach() in TastSetManagerSuite and
TaskSchedulerImplSuite
## What changes were proposed in this pull request?
In the `afterEach()` method of both `TastSetManagerSuite` and
`TaskSchedulerImplSuite`, `super.afterEach()` shall be called at the end,
because it shall stop the SparkContext.
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93706/testReport/org.apache.spark.scheduler/TaskSchedulerImplSuite/_It_is_not_a_test_it_is_a_sbt_testing_SuiteSelector_/
The test failure is caused by the above reason, the newly added
`barrierCoordinator` required `rpcEnv` which has been stopped before
`TaskSchedulerImpl` doing cleanup.
## How was this patch tested?
Existing tests.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jiangxb1987/spark afterEach
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/21908.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #21908
----
commit 0bb6a5b29beb987fdaac2d595cfd63fbd752af65
Author: Xingbo Jiang <xingbo.jiang@...>
Date: 2018-07-28T16:09:07Z
fix afterEach
----
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
