Skip to content

Commit

Permalink
[SPARK-6980] Missed one usage of deprecated RpcUtils.askTimeout in Ya…
Browse files Browse the repository at this point in the history
…rnSchedulerBackend although it is not being used, and fixed SparkConfSuite UT to not use deprecated RpcUtils functions
  • Loading branch information
BryanCutler committed Jun 30, 2015
1 parent 6a1c50d commit 4e89c75
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ private[spark] abstract class YarnSchedulerBackend(
private val yarnSchedulerEndpoint = rpcEnv.setupEndpoint(
YarnSchedulerBackend.ENDPOINT_NAME, new YarnSchedulerEndpoint(rpcEnv))

private implicit val askTimeout = RpcUtils.askTimeout(sc.conf)
private implicit val askTimeout = RpcUtils.askRpcTimeout(sc.conf)

/**
* Request executors from the ApplicationMaster by specifying the total number desired.
Expand Down
4 changes: 2 additions & 2 deletions core/src/test/scala/org/apache/spark/SparkConfSuite.scala
Original file line number Diff line number Diff line change
Expand Up @@ -260,10 +260,10 @@ class SparkConfSuite extends SparkFunSuite with LocalSparkContext with ResetSyst
assert(RpcUtils.retryWaitMs(conf) === 2L)

conf.set("spark.akka.askTimeout", "3")
assert(RpcUtils.askTimeout(conf) === (3 seconds))
assert(RpcUtils.askRpcTimeout(conf).duration === (3 seconds))

conf.set("spark.akka.lookupTimeout", "4")
assert(RpcUtils.lookupTimeout(conf) === (4 seconds))
assert(RpcUtils.lookupRpcTimeout(conf).duration === (4 seconds))
}
}

Expand Down

0 comments on commit 4e89c75

Please sign in to comment.