Skip to content

Commit 996332a

Browse files
committed
Add types so that tests compile
1 parent a46144a commit 996332a

File tree

2 files changed

+9
-9
lines changed

2 files changed

+9
-9
lines changed

sql/core/src/test/scala/org/apache/spark/sql/execution/SortSuite.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ class SortSuite extends SparkPlanTest {
3737

3838
checkAnswer(
3939
input,
40-
child => new ExternalSort(sortOrder, global = false, child),
40+
(child: SparkPlan) => new ExternalSort(sortOrder, global = false, child),
4141
input.sorted
4242
)
4343

sql/core/src/test/scala/org/apache/spark/sql/execution/SparkPlanTest.scala

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -26,16 +26,16 @@ import org.apache.spark.sql.{Row, DataFrame}
2626
import org.apache.spark.sql.catalyst.util._
2727

2828
/**
29-
* Base class for writing tests for individual physical operators. For an example of how this class
30-
* can be used, see [[SortSuite]].
29+
* Base class for writing tests for individual physical operators. For an example of how this
30+
* class's test helper methods can be used, see [[SortSuite]].
3131
*/
3232
class SparkPlanTest extends SparkFunSuite {
3333

3434
/**
3535
* Runs the plan and makes sure the answer matches the expected result.
3636
* @param input the input data to be used.
37-
* @param planFunction a function which accepts the input SparkPlan and uses it to instantiate the
38-
* physical operator that's being tested.
37+
* @param planFunction a function which accepts the input SparkPlan and uses it to instantiate
38+
* the physical operator that's being tested.
3939
* @param expectedAnswer the expected result in a [[Seq]] of [[Row]]s.
4040
*/
4141
protected def checkAnswer(
@@ -51,8 +51,8 @@ class SparkPlanTest extends SparkFunSuite {
5151
/**
5252
* Runs the plan and makes sure the answer matches the expected result.
5353
* @param input the input data to be used.
54-
* @param planFunction a function which accepts the input SparkPlan and uses it to instantiate the
55-
* physical operator that's being tested.
54+
* @param planFunction a function which accepts the input SparkPlan and uses it to instantiate
55+
* the physical operator that's being tested.
5656
* @param expectedAnswer the expected result in a [[Seq]] of [[Product]]s.
5757
*/
5858
protected def checkAnswer[A <: Product : TypeTag](
@@ -76,8 +76,8 @@ object SparkPlanTest {
7676
/**
7777
* Runs the plan and makes sure the answer matches the expected result.
7878
* @param input the input data to be used.
79-
* @param planFunction a function which accepts the input SparkPlan and uses it to instantiate the
80-
* physical operator that's being tested.
79+
* @param planFunction a function which accepts the input SparkPlan and uses it to instantiate
80+
* the physical operator that's being tested.
8181
* @param expectedAnswer the expected result in a [[Seq]] of [[Row]]s.
8282
*/
8383
def checkAnswer(

0 commit comments

Comments
 (0)