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

    https://github.com/apache/spark/pull/20632#discussion_r171982559
  
    --- Diff: 
mllib/src/test/scala/org/apache/spark/ml/tree/impl/RandomForestSuite.scala ---
    @@ -703,4 +707,16 @@ private object RandomForestSuite {
         val (indices, values) = map.toSeq.sortBy(_._1).unzip
         Vectors.sparse(size, indices.toArray, values.toArray)
       }
    +
    +  @tailrec
    +  private def getSumLeafCounters(nodes: List[Node], acc: Long = 0): Long =
    --- End diff --
    
    Need to enclose the function body in curly braces


---

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

Reply via email to