Github user dongjoon-hyun commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22499#discussion_r219685155
  
    --- Diff: 
mllib/src/test/scala/org/apache/spark/mllib/linalg/UDTSerializationBenchmark.scala
 ---
    @@ -18,52 +18,52 @@
     package org.apache.spark.mllib.linalg
     
     import org.apache.spark.sql.catalyst.encoders.ExpressionEncoder
    -import org.apache.spark.util.Benchmark
    +import org.apache.spark.util.{Benchmark, BenchmarkBase => 
FileBenchmarkBase}
     
     /**
      * Serialization benchmark for VectorUDT.
    + * To run this benchmark:
    + * 1. without sbt: bin/spark-submit --class <this class> <spark mllib test 
jar>
    --- End diff --
    
    +1 for fix the docs to pass Jenkins.
    Also, could you rebase this PR to resolve conflicts, @seancxmao ?


---

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

Reply via email to