Github user mgaido91 commented on the issue: https://github.com/apache/spark/pull/22087 @srowen I think it would not be a big deal if we would change the return type of these to Double, since they are now private. If that is the only change needed we may be able to reuse the `PredictionModel` also for clustering models. But if we expose them, going back would be harder. So I'd first agree on whether to try or not to make them `PredictionModel`s: if not, this PR is good IMHO.
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org