diff options
author | Bryan Cutler <cutlerb@gmail.com> | 2017-04-03 10:56:54 +0200 |
---|---|---|
committer | Nick Pentreath <nickp@za.ibm.com> | 2017-04-03 10:56:54 +0200 |
commit | 2a903a1eec46e3bd58af0fcbc57e76752d9c18b3 (patch) | |
tree | 9080b7736edf130303043d56700a0f08337e6d64 /scalastyle-config.xml | |
parent | 93dbfe705f3e7410a7267e406332ffb3c3077829 (diff) | |
download | spark-2a903a1eec46e3bd58af0fcbc57e76752d9c18b3.tar.gz spark-2a903a1eec46e3bd58af0fcbc57e76752d9c18b3.tar.bz2 spark-2a903a1eec46e3bd58af0fcbc57e76752d9c18b3.zip |
[SPARK-19985][ML] Fixed copy method for some ML Models
## What changes were proposed in this pull request?
Some ML Models were using `defaultCopy` which expects a default constructor, and others were not setting the parent estimator. This change fixes these by creating a new instance of the model and explicitly setting values and parent.
## How was this patch tested?
Added `MLTestingUtils.checkCopy` to the offending models to tests to verify the copy is made and parent is set.
Author: Bryan Cutler <cutlerb@gmail.com>
Closes #17326 from BryanCutler/ml-model-copy-error-SPARK-19985.
Diffstat (limited to 'scalastyle-config.xml')
0 files changed, 0 insertions, 0 deletions