Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/19893#discussion_r155393714 --- 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 -- I don't know what that is. The import order is described in http://spark.apache.org/contributing.html, section "Imports".
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org