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

Feynman Liang edited comment on SPARK-9660 at 8/7/15 8:04 PM:
--------------------------------------------------------------

Should [RandomForestClassificationModel's aux 
constructor|https://github.com/apache/spark/blob/master/mllib/src/main/scala/org/apache/spark/ml/classification/RandomForestClassifier.scala#L139]
 be private? Ditto for 
[DecisionTreeRegressionModel|https://github.com/apache/spark/blob/master/mllib/src/main/scala/org/apache/spark/ml/regression/DecisionTreeRegressor.scala#L110],
 
[RandomForestRegressionModel|https://github.com/apache/spark/blob/master/mllib/src/main/scala/org/apache/spark/ml/regression/DecisionTreeRegressor.scala#L110]


was (Author: fliang):
Should [RandomForestClassificationModel's aux 
constructor|https://github.com/apache/spark/blob/master/mllib/src/main/scala/org/apache/spark/ml/classification/RandomForestClassifier.scala#L139]
 be private? Ditto for 
[DecisionTreeRegressionModel|https://github.com/apache/spark/blob/master/mllib/src/main/scala/org/apache/spark/ml/regression/DecisionTreeRegressor.scala#L110]

> ML 1.5 QA: API: New Scala APIs, docs
> ------------------------------------
>
>                 Key: SPARK-9660
>                 URL: https://issues.apache.org/jira/browse/SPARK-9660
>             Project: Spark
>          Issue Type: Sub-task
>          Components: Documentation, ML, MLlib
>            Reporter: Joseph K. Bradley
>
> Audit new public Scala APIs added to MLlib.  Take note of:
> * Protected/public classes or methods.  If access can be more private, then 
> it should be.
> * Also look for non-sealed traits.
> * Documentation: Missing?  Bad links or formatting?
> *Make sure to check the object doc!*
> As you find issues, please comment here, or better yet create JIRAs and link 
> them.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to