Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20394#discussion_r163799202
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/joins/BroadcastJoinSuite.scala
---
@@ -70,8 +71,8 @@ class BroadcastJoinSuite extends QueryTest with
SQLTestUtils {
private def testBroadcastJoin[T: ClassTag](
joinType: String,
forceBroadcast: Boolean = false): SparkPlan = {
- val df1 = spark.createDataFrame(Seq((1, "4"), (2, "2"))).toDF("key",
"value")
- val df2 = spark.createDataFrame(Seq((1, "1"), (2, "2"))).toDF("key",
"value")
+ val df1 = Seq((1, "4"), (2, "2")).toDF("key", "value")
+ val df2 = Seq((1, "1"), (2, "2")).toDF("key", "value")
--- End diff --
some code style fixing
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
