Skip to content

Commit

Permalink
[SPARK-22222][CORE][TEST][FOLLOW-UP] Remove redundant and deprecated …
Browse files Browse the repository at this point in the history
…`Timeouts`

## What changes were proposed in this pull request?

Since SPARK-21939, Apache Spark uses `TimeLimits` instead of the deprecated `Timeouts`. This PR fixes the build warning `BufferHolderSparkSubmitSuite.scala` introduced at [SPARK-22222](https://github.com/apache/spark/pull/19460/files#diff-d8cf6e0c229969db94ec8ffc31a9239cR36) by removing the redundant `Timeouts`.
```scala
trait Timeouts in package concurrent is deprecated: Please use org.scalatest.concurrent.TimeLimits instead
[warn]     with Timeouts {
```
## How was this patch tested?

N/A

Author: Dongjoon Hyun <[email protected]>

Closes apache#19697 from dongjoon-hyun/SPARK-22222.
  • Loading branch information
dongjoon-hyun authored and HyukjinKwon committed Nov 9, 2017
1 parent 695647b commit 98be55c
Showing 1 changed file with 1 addition and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@
package org.apache.spark.sql.catalyst.expressions.codegen

import org.scalatest.{BeforeAndAfterEach, Matchers}
import org.scalatest.concurrent.Timeouts

import org.apache.spark.{SparkFunSuite, TestUtils}
import org.apache.spark.deploy.SparkSubmitSuite
Expand All @@ -32,8 +31,7 @@ class BufferHolderSparkSubmitSuite
extends SparkFunSuite
with Matchers
with BeforeAndAfterEach
with ResetSystemProperties
with Timeouts {
with ResetSystemProperties {

test("SPARK-22222: Buffer holder should be able to allocate memory larger than 1GB") {
val unusedJar = TestUtils.createJarWithClasses(Seq.empty)
Expand Down

0 comments on commit 98be55c

Please sign in to comment.