[ 
https://issues.apache.org/jira/browse/SPARK-30377?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17005086#comment-17005086
 ] 

Huaxin Gao commented on SPARK-30377:
------------------------------------

Seems there is no API difference, but it would be more logical to make every 
Regressor to extend ml.Regressor.

I quickly checked on Classifiers. Everything extends Classifier except 
OneVsRest. I guess  I will open a separate Jira to make OneVsRest extend 
Classifier. 

> Make Regressors extend abstract class Regressor
> -----------------------------------------------
>
>                 Key: SPARK-30377
>                 URL: https://issues.apache.org/jira/browse/SPARK-30377
>             Project: Spark
>          Issue Type: Improvement
>          Components: ML
>    Affects Versions: 3.0.0
>            Reporter: zhengruifeng
>            Priority: Major
>
> Just found that  {{AFTSurvivalRegression}} , {{DecisionTreeRegressor}}, 
> {{FMRegressor}}, {{GBTRegressor}}, {{RandomForestRegressor}} directly extend 
> {{Predictor}}
>  
> Only {{GeneralizedLinearRegression}} and {{LinearRegression now extend 
> Regressor.}} 
>  
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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

Reply via email to