Skip to content

[SPARK-13153][PySpark] ML persistence failed when handle no default value parameter - #11043

Closed
Wenpei wants to merge 3 commits into
apache:masterfrom
Wenpei:spark-13153-handle-param-withnodefaultvalue
Closed

[SPARK-13153][PySpark] ML persistence failed when handle no default value parameter#11043
Wenpei wants to merge 3 commits into
apache:masterfrom
Wenpei:spark-13153-handle-param-withnodefaultvalue

Conversation

@Wenpei

Copy link
Copy Markdown
Contributor

Fix this defect by check default value exist or not.

@yanboliang Please help to review.

Comment threadpython/pyspark/ml/wrapper.py Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should check by isDefined rather than hasDefault here.

@yanboliang

Copy link
Copy Markdown
Contributor

ping @mengxr@jkbradley
Could you add @Wenpei to white list ? This is an obvious bug and we should fix it.

@Wenpei

Copy link
Copy Markdown
ContributorAuthor

It's OK now and ready to launch test.

@mengxr

Copy link
Copy Markdown
Contributor

ok to test

@mengxr

Copy link
Copy Markdown
Contributor

LGTM pending Jenins

@SparkQA

Copy link
Copy Markdown

Test build #51161 has finished for PR 11043 at commit 06e06f7.

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

asfgit pushed a commit that referenced this pull request Feb 12, 2016
…alue parameter
Fix this defect by check default value exist or not.
yanboliang Please help to review.
Author: Tommy YU <tummyyu@163.com>
Closes#11043 from Wenpei/spark-13153-handle-param-withnodefaultvalue.
(cherry picked from commit d3e2e20)
Signed-off-by: Xiangrui Meng <meng@databricks.com>
@mengxr

Copy link
Copy Markdown
Contributor

Merged into master and branch-1.6. Thanks!

@Wenpei
Wenpei deleted the spark-13153-handle-param-withnodefaultvalue branch February 15, 2016 03:03
Sign up for freeto 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.

4 participants

@Wenpei@yanboliang@mengxr@SparkQA