Github user imatiach-msft commented on a diff in the pull request:

    https://github.com/apache/spark/pull/17086#discussion_r182303815
  
    --- Diff: 
mllib/src/main/scala/org/apache/spark/mllib/evaluation/MulticlassMetrics.scala 
---
    @@ -27,10 +27,11 @@ import org.apache.spark.sql.DataFrame
     /**
      * Evaluator for multiclass classification.
      *
    - * @param predictionAndLabels an RDD of (prediction, label) pairs.
    + * @param predAndLabelsWithOptWeight an RDD of (prediction, label, weight) 
or
    + *                                   (prediction, label) pairs.
      */
     @Since("1.1.0")
    -class MulticlassMetrics @Since("1.1.0") (predictionAndLabels: RDD[(Double, 
Double)]) {
    +class MulticlassMetrics @Since("1.1.0") (predAndLabelsWithOptWeight: 
RDD[_]) {
    --- End diff --
    
    hmm the build fails here though with an error indicating that the methods 
are the same after type erasure, perhaps I should revert this code back:
    
    [error] 
/home/jenkins/workspace/SparkPullRequestBuilder@2/mllib/src/main/scala/org/apache/spark/mllib/evaluation/MulticlassMetrics.scala:35:
 double definition:
    [error] constructor MulticlassMetrics: (predLabelsWeight: 
org.apache.spark.rdd.RDD[(Double, Double, 
Double)])org.apache.spark.mllib.evaluation.MulticlassMetrics at line 33 and
    [error] constructor MulticlassMetrics: (predAndLabels: 
org.apache.spark.rdd.RDD[(Double, 
Double)])org.apache.spark.mllib.evaluation.MulticlassMetrics at line 35
    [error] have same type after erasure: (predLabelsWeight: 
org.apache.spark.rdd.RDD)org.apache.spark.mllib.evaluation.MulticlassMetrics
    [error]   def this(predAndLabels: RDD[(Double, Double)]) =
    



---

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

Reply via email to