GitHub user wojtek-szymanski opened a pull request:
https://github.com/apache/spark/pull/15759
Bug fix: Pipeline.copy does not create an instance with the same UID
## What changes were proposed in this pull request?
Motivation:
`org.apache.spark.ml.Pipeline.copy(extra: ParamMap)` does not create an
instance with the same UID. It does not conform to the method specification
from its base class `org.apache.spark.ml.param.Params.copy(extra: ParamMap)`
Solution:
- fix for Pipeline UID
- introduced new tests for `org.apache.spark.ml.Pipeline.copy`
- minor improvements in test for `org.apache.spark.ml.PipelineModel.copy`
## How was this patch tested?
Introduced new unit test:
`org.apache.spark.ml.PipelineSuite."Pipeline.copy"`
Improved existing unit test:
`org.apache.spark.ml.PipelineSuite."PipelineModel.copy"`
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/wojtek-szymanski/spark SPARK-18210
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/15759.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #15759
----
commit 8764e36f9764f89343a3e0fe6eff05cb41bb36cf
Author: Wojciech Szymanski <[email protected]>
Date: 2016-11-01T14:11:20Z
Bug fix: Pipeline.copy does not create an instance with the same UID
----
---
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]
