Skip to content
This repository was archived by the owner on Jan 9, 2020. It is now read-only.

Commit 653e6f1

Browse files
jerryshaocloud-fan
authored andcommitted
[SPARK-12552][FOLLOWUP] Fix flaky test for "o.a.s.deploy.master.MasterSuite.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 <[email protected]> Closes apache#18321 from jerryshao/SPARK-12552-followup. (cherry picked from commit 2837b14) Signed-off-by: Wenchen Fan <[email protected]>
1 parent 9909be3 commit 653e6f1

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

core/src/test/scala/org/apache/spark/deploy/master/MasterSuite.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -169,7 +169,7 @@ class MasterSuite extends SparkFunSuite
169169
master.rpcEnv.setupEndpoint(Master.ENDPOINT_NAME, master)
170170
// Wait until Master recover from checkpoint data.
171171
eventually(timeout(5 seconds), interval(100 milliseconds)) {
172-
master.idToApp.size should be(1)
172+
master.workers.size should be(1)
173173
}
174174

175175
master.idToApp.keySet should be(Set(fakeAppInfo.id))

0 commit comments

Comments
 (0)