Github user jkbradley commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19208#discussion_r148885486
  
    --- Diff: 
mllib/src/main/scala/org/apache/spark/ml/tuning/CrossValidator.scala ---
    @@ -271,6 +303,20 @@ class CrossValidatorModel private[ml] (
     @Since("1.6.0")
     object CrossValidatorModel extends MLReadable[CrossValidatorModel] {
     
    +  private[CrossValidatorModel] def copySubModels(subModels: 
Option[Array[Array[Model[_]]]]) = {
    +    subModels.map { subModels =>
    --- End diff --
    
    Can this be simplified using map?
    ```
    subModels.map(_.map(_.map(_.copy(...).asInstanceOf[...])))
    ```


---

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

Reply via email to