You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@spark.apache.org by "Maciej Szymkiewicz (Jira)" <ji...@apache.org> on 2019/10/05 14:00:00 UTC
[jira] [Created] (SPARK-29363) o.a.s.ml.regression.Regressor should
be public
Maciej Szymkiewicz created SPARK-29363:
------------------------------------------
Summary: o.a.s.ml.regression.Regressor should be public
Key: SPARK-29363
URL: https://issues.apache.org/jira/browse/SPARK-29363
Project: Spark
Issue Type: Improvement
Components: ML
Affects Versions: 2.4.4, 2.3.0, 3.0.0
Reporter: Maciej Szymkiewicz
At the moment [{{o.a.s.ml.regression.Regressor}}|https://github.com/apache/spark/blob/8556710409d9f2fbaee9dbf76a2ea70218316693/mllib/src/main/scala/org/apache/spark/ml/regression/Regressor.scala#L31] is marked as {{private[spark]}}.
This however is inconsistent with the rest of the API:
* Similarly positioned {{o.a.s.ml.classification.Classifier}} [is public|https://github.com/apache/spark/blob/7955b3962ac46b89564e0613db7bea98a1478bf2/mllib/src/main/scala/org/apache/spark/ml/classification/Classifier.scala#L57-L58].
* Accompanying {{o.a.s.regression.RegressionModel}} [is public|https://github.com/apache/spark/blob/8556710409d9f2fbaee9dbf76a2ea70218316693/mllib/src/main/scala/org/apache/spark/ml/regression/Regressor.scala#L49]
Additionally it hides useful information from the end user, who might be interested in defining methods which operate specifically on {{Regressors}} (should other regressor-like {{Estimators}} extend {{Regressor}} as well? Let's say {{DecisionTreeRegressor}}).
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@spark.apache.org
For additional commands, e-mail: issues-help@spark.apache.org