Skip to content

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 <[email protected]>

Closes #8637 from yanboliang/spark-10470.

(cherry picked from commit f7b55db)
Signed-off-by: Xiangrui Meng <[email protected]>
@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 <[email protected]>

Closes apache#8637 from yanboliang/spark-10470.

(cherry picked from commit f7b55db)
Signed-off-by: Xiangrui Meng <[email protected]>
(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

Development

Successfully merging this pull request may close these issues.

3 participants