[SPARK-29363] o.a.s.ml.regression.Regressor should be public

• Type: Improvement
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 2.3.0, 2.4.4, 3.0.0
• Fix Version/s: 3.0.0
• Component/s: ML
• Assignee: Maciej Szymkiewicz
• Reporter: Maciej Szymkiewicz
• Created: 05/Oct/19 13:59
• Updated: 06/Oct/19 08:27
• Resolved: 06/Oct/19 01:17

Description

At the moment o.a.s.ml.regression.Regressor 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.
• Accompanying o.a.s.regression.RegressionModel is public

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).

Issue Links

• links to: GitHub Pull Request #26033