Github user smurakozi commented on a diff in the pull request:

    https://github.com/apache/spark/pull/20121#discussion_r162058792
  
    --- Diff: 
mllib/src/test/scala/org/apache/spark/ml/classification/OneVsRestSuite.scala ---
    @@ -17,14 +17,13 @@
     
     package org.apache.spark.ml.classification
     
    -import org.apache.spark.SparkFunSuite
     import org.apache.spark.ml.attribute.NominalAttribute
     import org.apache.spark.ml.classification.LogisticRegressionSuite._
     import org.apache.spark.ml.feature.LabeledPoint
     import org.apache.spark.ml.feature.StringIndexer
    -import org.apache.spark.ml.linalg.Vectors
    +import org.apache.spark.ml.linalg.{Vector, Vectors}
     import org.apache.spark.ml.param.{ParamMap, ParamsSuite}
    -import org.apache.spark.ml.util.{DefaultReadWriteTest, MetadataUtils, 
MLTestingUtils}
    +import org.apache.spark.ml.util.{DefaultReadWriteTest, MetadataUtils, 
MLTest, MLTestingUtils}
    --- End diff --
    
    nit:  import org.apache.spark.mllib.util.MLlibTestSparkContext
    is unused at line 32


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to