[SPARK-13010] [ML] [SparkR] Implement a simple wrapper of AFTSurvivalRegression in SparkR

[yanboliang]

What changes were proposed in this pull request?

This PR continues the work in #11447, we implemented the wrapper of AFTSurvivalRegression named survreg in SparkR.

How was this patch tested?

Test against output from R package survival's survreg.

cc @mengxr @felixcheung

Close #11447

[SparkQA]

Test build #54031 has finished for PR 11932 at commit e6cc821.

• This patch fails SparkR unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #54038 has finished for PR 11932 at commit dc64aaa.

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

[yanboliang]

This test case is for review only, to show an example of SparkR::survreg vs survival::survreg. Will remove if merged.

[mengxr]

@yanboliang The implementation looks good to me. Please update the PR and remove the survreg2 test.

[felixcheung]

Right - it's nice to see if we have similar interface.

This test we are keeping, it passes locally when you have survival package as

model <- survreg(formula = survival::Surv(time, status) ~ x + sex, data = rData)

ie. without survival:: namespace in front of survreg?

It is preferred that we don't mask/conflict with other packages if possible.

[mengxr]

@felixcheung It works without survival:: because survreg is a regular function, like glm. If it is an S3 generic method like naiveBayes, we have to use the namespace prefix.

[SparkQA]

Test build #54139 has finished for PR 11932 at commit 9023a92.

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

[mengxr]

LGTM. Merged into master. Thanks!