Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/19208#discussion_r148859043
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tuning/CrossValidator.scala ---
@@ -252,19 +252,29 @@ object CrossValidator extends
MLReadable[CrossValidator] {
class CrossValidatorModel private[ml] (
@Since("1.4.0") override val uid: String,
@Since("1.2.0") val bestModel: Model[_],
- @Since("1.5.0") val avgMetrics: Array[Double],
- @Since("2.3.0") val subModels: Option[Array[Array[Model[_]]]])
+ @Since("1.5.0") val avgMetrics: Array[Double])
extends Model[CrossValidatorModel] with CrossValidatorParams with
MLWritable {
/** A Python-friendly auxiliary constructor. */
private[ml] def this(uid: String, bestModel: Model[_], avgMetrics:
JList[Double]) = {
- this(uid, bestModel, avgMetrics.asScala.toArray, null)
+ this(uid, bestModel, avgMetrics.asScala.toArray)
}
- private[ml] def this(uid: String, bestModel: Model[_], avgMetrics:
Array[Double]) = {
- this(uid, bestModel, avgMetrics, null)
+ private var _subModels: Option[Array[Array[Model[_]]]] = None
+
+ @Since("2.3.0")
--- End diff --
Only use Since annotations for public APIs
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
