[GitHub] spark issue #18980: Correct validateAndTransformSchema in GaussianMixture

2017-08-20 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18980 **[Test build #3892 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3892/testReport)** for PR 18980 at commit

[GitHub] spark issue #18980: Correct validateAndTransformSchema in GaussianMixture

2017-08-20 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/18980 I started tests. One last request, start the title with `[MINOR]`. See https://spark.apache.org/contributing.html --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark issue #18980: Correct validateAndTransformSchema in GaussianMixture

2017-08-20 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18980 **[Test build #3892 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3892/testReport)** for PR 18980 at commit

[GitHub] spark issue #18980: Correct validateAndTransformSchema in GaussianMixture

2017-08-18 Thread sharp-pixel
Github user sharp-pixel commented on the issue: https://github.com/apache/spark/pull/18980 I just fixed AFTSurvivalRegression also. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark issue #18980: Correct validateAndTransformSchema in GaussianMixture

2017-08-18 Thread yanboliang
Github user yanboliang commented on the issue: https://github.com/apache/spark/pull/18980 +1 @srowen, this is a bug. @sharp-pixel Would you mind to fix both ```GaussianMixture``` and ```AFTSurvivalRegression```? It's better to file a JIRA firstly and add some unit tests. Thanks.

[GitHub] spark issue #18980: Correct validateAndTransformSchema in GaussianMixture

2017-08-18 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/18980 Yeah I think you're right. There's a similar issue in `AFTSurvivalRegressionParams` CC @yanboliang --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark issue #18980: Correct validateAndTransformSchema in GaussianMixture

2017-08-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18980 Can one of the admins verify this patch? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this