Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-10470] [ML] ml.IsotonicRegressionModel.copy should set parent #8637

Closed
wants to merge 1 commit into from

Conversation

yanboliang
Copy link
Contributor

Copied model must have the same parent, but ml.IsotonicRegressionModel.copy did not set parent.
Here fix it and add test case.

@SparkQA
Copy link

SparkQA commented Sep 7, 2015

Test build #42083 has finished for PR 8637 at commit 20717d8.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@asfgit asfgit closed this in f7b55db Sep 8, 2015
asfgit pushed a commit that referenced this pull request Sep 8, 2015
Copied model must have the same parent, but ml.IsotonicRegressionModel.copy did not set parent.
Here fix it and add test case.

Author: Yanbo Liang <ybliang8@gmail.com>

Closes #8637 from yanboliang/spark-10470.

(cherry picked from commit f7b55db)
Signed-off-by: Xiangrui Meng <meng@databricks.com>
@mengxr
Copy link
Contributor

mengxr commented Sep 8, 2015

LGTM. Merged into master and branch-1.5. Thanks!

@yanboliang yanboliang deleted the spark-10470 branch September 9, 2015 02:12
ashangit pushed a commit to ashangit/spark that referenced this pull request Oct 19, 2016
Copied model must have the same parent, but ml.IsotonicRegressionModel.copy did not set parent.
Here fix it and add test case.

Author: Yanbo Liang <ybliang8@gmail.com>

Closes apache#8637 from yanboliang/spark-10470.

(cherry picked from commit f7b55db)
Signed-off-by: Xiangrui Meng <meng@databricks.com>
(cherry picked from commit 34d417e)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants