srowen commented on a change in pull request #27600: [MINOR][ML] ML cleanup
URL: https://github.com/apache/spark/pull/27600#discussion_r380225303
##########
File path:
mllib/src/main/scala/org/apache/spark/ml/classification/RandomForestClassifier.scala
##########
@@ -256,7 +256,7 @@ class RandomForestClassificationModel private[ml] (
// Classifies using majority votes.
// Ignore the tree weights since all are 1.0 for now.
val votes = Array.ofDim[Double](numClasses)
- _trees.view.foreach { tree =>
+ _trees.foreach { tree =>
Review comment:
In some cases I think we used view to avoid copying the result of a map or
something, but I don't think that's the case here. It's an Array.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
