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

Reply via email to