diff options
| author | Yanbo Liang <ybliang8@gmail.com> | 2016-09-21 20:14:18 -0700 |
|---|---|---|
| committer | Yanbo Liang <ybliang8@gmail.com> | 2016-09-21 20:14:18 -0700 |
| commit | 6902edab7e80e96e3f57cf80f26cefb209d4d63c (patch) | |
| tree | 0db78d50af587d220f8e1dc7250c5eca022b0cbf /sql/core/src | |
| parent | c133907c5d9a6e6411b896b5e0cff48b2beff09f (diff) | |
| download | spark-6902edab7e80e96e3f57cf80f26cefb209d4d63c.tar.gz spark-6902edab7e80e96e3f57cf80f26cefb209d4d63c.tar.bz2 spark-6902edab7e80e96e3f57cf80f26cefb209d4d63c.zip | |
[SPARK-17315][FOLLOW-UP][SPARKR][ML] Fix print of Kolmogorov-Smirnov test summary
## What changes were proposed in this pull request?
#14881 added Kolmogorov-Smirnov Test wrapper to SparkR. I found that ```print.summary.KSTest``` was implemented inappropriately and result in no effect.
Running the following code for KSTest:
```Scala
data <- data.frame(test = c(0.1, 0.15, 0.2, 0.3, 0.25, -1, -0.5))
df <- createDataFrame(data)
testResult <- spark.kstest(df, "test", "norm")
summary(testResult)
```
Before this PR:
After this PR:
The new implementation is similar with [```print.summary.GeneralizedLinearRegressionModel```](https://github.com/apache/spark/blob/master/R/pkg/R/mllib.R#L284) of SparkR and [```print.summary.glm```](https://svn.r-project.org/R/trunk/src/library/stats/R/glm.R) of native R.
BTW, I removed the comparison of ```print.summary.KSTest``` in unit test, since it's only wrappers of the summary output which has been checked. Another reason is that these comparison will output summary information to the test console, it will make the test output in a mess.
## How was this patch tested?
Existing test.
Author: Yanbo Liang <ybliang8@gmail.com>
Closes #15139 from yanboliang/spark-17315.
Diffstat (limited to 'sql/core/src')
0 files changed, 0 insertions, 0 deletions