GitHub user shkdee opened a pull request:
https://github.com/apache/spark/pull/12415
[BUG] LinearRegressionModel use default parameters if yStd is 0
## What changes were proposed in this pull request?
In the (rare) case where yStd is 0 in LinearRegression, parameters are not
copied immediately to the created LinearRegressionModel instance, but they
should (as in all other cases), because now the default column names are used
in and returned by model.findSummaryModelAndPredictionCol(), which leads to
schema validation errors.
## How was this patch tested?
some simple manual tests
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/shkdee/spark patch-1
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/12415.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 #12415
----
commit c05db63f1b6af0bdc6f0c277679e53a01913b3c5
Author: Louis Traynard <shk...@users.noreply.github.com>
Date: 2016-04-15T10:06:01Z
[BUG] copy parameters to LinearRegressionModel instance
In the (rare) case where yStd is 0 in LinearRegression, parameters are not
copied immediately to the created LinearRegressionModel instance, but they
should (as in all other cases), because now the default column names are used
in and returned by model.findSummaryModelAndPredictionCol(), which leads to
schema validation errors.
----
---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
