Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15652#discussion_r87373535
--- Diff: mllib/src/main/scala/org/apache/spark/ml/tree/treeParams.scala ---
@@ -85,8 +85,10 @@ private[ml] trait DecisionTreeParams extends
PredictorParams
* (default = 256 MB)
* @group expertParam
*/
- final val maxMemoryInMB: IntParam = new IntParam(this, "maxMemoryInMB",
- "Maximum memory in MB allocated to histogram aggregation.",
+ final val maxMemoryInMB: IntParam = new IntParam(this, "maxMemoryInMB",
"Maximum memory in MB" +
--- End diff --
Yes, it should not be in this pull request because it's not at all related.
Sometimes it makes sense to touch up code while changing it for other reasons
but that's not the case here.
I think these changes may be too insignificant for their own PR. If you can
review a module for typos, that would be better to do it all at once.
For this particular change, I don't know if the parameter's help message
needs to have all its documentation. It's just describing the basic nature of
the parameter.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
