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

    https://github.com/apache/spark/pull/20964#discussion_r178778285
  
    --- Diff: 
mllib/src/test/scala/org/apache/spark/ml/feature/ImputerSuite.scala ---
    @@ -76,6 +75,28 @@ class ImputerSuite extends SparkFunSuite with 
MLlibTestSparkContext with Default
         ImputerSuite.iterateStrategyTest(imputer, df)
       }
     
    +  test("Imputer should work with Structured Streaming") {
    +    val localSpark = spark
    +    import localSpark.implicits._
    +    val df = Seq[(java.lang.Double, Double)](
    +      (4.0, 4.0),
    +      (10.0, 10.0),
    +      (10.0, 10.0),
    +      (Double.NaN, 8.0),
    +      (null, 8.0)
    +    ).toDF("value", "expected_mean_value")
    --- End diff --
    
    Why the "value" column use `java.lang.Double` type ?


---

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

Reply via email to