Skip to content

Commit

Permalink
[SPARK-45245][CONNECT][TESTS][FOLLOW-UP] Remove unneeded Matchers tra…
Browse files Browse the repository at this point in the history
…it in the test

### What changes were proposed in this pull request?

This PR is a followup of apache#43023 that addresses a post-review comment.

### Why are the changes needed?

It is unnecessary. It also matters with Scala compatibility so should better remove if unused.

### Does this PR introduce _any_ user-facing change?

No, test-only.

### How was this patch tested?

Manually.

### Was this patch authored or co-authored using generative AI tooling?

No.

Closes apache#45459 from HyukjinKwon/SPARK-45245-folllowup.

Lead-authored-by: Hyukjin Kwon <[email protected]>
Co-authored-by: Hyukjin Kwon <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
  • Loading branch information
2 people authored and dongjoon-hyun committed Mar 11, 2024
1 parent 610840e commit f97da16
Showing 1 changed file with 1 addition and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -25,15 +25,13 @@ import scala.concurrent.ExecutionContext.Implicits.global
import scala.concurrent.Future
import scala.concurrent.duration._

import org.scalatest.matchers.must.Matchers

import org.apache.spark.SharedSparkContext
import org.apache.spark.SparkException
import org.apache.spark.SparkFunSuite
import org.apache.spark.util.ThreadUtils

// Tests for PythonWorkerFactory.
class PythonWorkerFactorySuite extends SparkFunSuite with Matchers with SharedSparkContext {
class PythonWorkerFactorySuite extends SparkFunSuite with SharedSparkContext {

test("createSimpleWorker() fails with a timeout error if worker does not connect back") {
// It verifies that server side times out in accept(), if the worker does not connect back.
Expand Down

0 comments on commit f97da16

Please sign in to comment.