Skip to content

Commit

Permalink
[SPARK-12552][FOLLOWUP] Fix flaky test for "o.a.s.deploy.master.Maste…
Browse files Browse the repository at this point in the history
…rSuite.master correctly recover the application"

## What changes were proposed in this pull request?

Due to the RPC asynchronous event processing, The test "correctly recover the application" could potentially be failed. The issue could be found in here: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78126/testReport/org.apache.spark.deploy.master/MasterSuite/master_correctly_recover_the_application/.

So here fixing this flaky test.

## How was this patch tested?

Existing UT.

CC cloud-fan jiangxb1987 , please help to review, thanks!

Author: jerryshao <sshao@hortonworks.com>

Closes #18321 from jerryshao/SPARK-12552-followup.
  • Loading branch information
jerryshao authored and cloud-fan committed Jun 16, 2017
1 parent 7a3e5dc commit 2837b14
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -214,7 +214,7 @@ class MasterSuite extends SparkFunSuite
master.rpcEnv.setupEndpoint(Master.ENDPOINT_NAME, master)
// Wait until Master recover from checkpoint data.
eventually(timeout(5 seconds), interval(100 milliseconds)) {
master.idToApp.size should be(1)
master.workers.size should be(1)
}

master.idToApp.keySet should be(Set(fakeAppInfo.id))
Expand Down

0 comments on commit 2837b14

Please sign in to comment.