Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19893#discussion_r155373350
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -20,10 +20,12 @@ package org.apache.spark
// scalastyle:off
import java.io.File
-import org.scalatest.{BeforeAndAfterAll, FunSuite, Outcome}
-
import org.apache.spark.internal.Logging
import org.apache.spark.util.AccumulatorContext
+import org.scalatest.{BeforeAndAfterAll, FunSuite, Outcome}
+
+import scala.collection.JavaConversions._
--- End diff --
Wonder why the style checker didn't complain, but `scala.*` imports should
be in the previous position.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
