Skip to content

Conversation

@yanboliang
Copy link
Contributor

@yanboliang yanboliang commented Jun 29, 2016

What changes were proposed in this pull request?

Fix wrong arguments description of survreg in SparkR.

How was this patch tested?

Arguments section of survreg doc before this PR (with wrong description for path and missing overwrite):
image

After this PR:
image

@yanboliang
Copy link
Contributor Author

cc @mengxr @junyangq

@SparkQA
Copy link

SparkQA commented Jun 29, 2016

Test build #61459 has finished for PR 13970 at commit b487efd.

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

@mengxr
Copy link
Contributor

mengxr commented Jun 29, 2016

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

asfgit pushed a commit that referenced this pull request Jun 29, 2016
## What changes were proposed in this pull request?
Fix wrong arguments description of ```survreg``` in SparkR.

## How was this patch tested?
```Arguments``` section of ```survreg``` doc before this PR (with wrong description for ```path``` and missing ```overwrite```):
![image](https://cloud.githubusercontent.com/assets/1962026/16447548/fe7a5ed4-3da1-11e6-8b96-b5bf2083b07e.png)

After this PR:
![image](https://cloud.githubusercontent.com/assets/1962026/16447617/368e0b18-3da2-11e6-8277-45640fb11859.png)

Author: Yanbo Liang <ybliang8@gmail.com>

Closes #13970 from yanboliang/spark-16143-followup.

(cherry picked from commit c6a220d)
Signed-off-by: Xiangrui Meng <meng@databricks.com>
@asfgit asfgit closed this in c6a220d Jun 29, 2016
@yanboliang yanboliang deleted the spark-16143-followup branch June 30, 2016 00:35
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