[SPARK-5214][Test] Add a test to demonstrate EventLoop can be stopped in the event thread

[zsxwing]

No description provided.

[SparkQA]

Test build #26009 has started for PR 4174 at commit 7aaa2d7.

• This patch merges cleanly.

[SparkQA]

Test build #26009 has finished for PR 4174 at commit 7aaa2d7.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
  • case class Rating(userId: Int, movieId: Int, rating: Float, timestamp: Long)
  • case class Movie(movieId: Int, title: String, genres: Seq[String])
  • case class Params(
  • class ALS extends Estimator[ALSModel] with ALSParams
  • case class RatingBlock(srcIds: Array[Int], dstIds: Array[Int], ratings: Array[Float])

[AmplabJenkins]

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/26009/
Test PASSed.

[srowen]

This was meant to go with #4016 ?

[zsxwing]

This was meant to go with #4016 ?

I didn't consider this case when doing #4016. Although #4016 happens to work, I still think it's worth to add a test to prevent later changes from breaking it.

[rxin]

can we reduce the check interval to like 5ms? Ideally unit tests should run in a very short amount of time. Having to wait 200ms is pretty long. Would be great to change rest of this file also.

[zsxwing]

Done

[SparkQA]

Test build #26049 has started for PR 4174 at commit 443e564.

• This patch merges cleanly.

[SparkQA]

Test build #26049 has finished for PR 4174 at commit 443e564.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[AmplabJenkins]

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/26049/
Test PASSed.

[rxin]

Thanks. Merging in master.