Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-215884375
Will do, thanks for pinging
---
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 h
Github user dbtsai commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-215588256
@jkbradley The `@Since` annotation was merged
https://github.com/apache/spark/pull/12416 Could you submit a followup PR?
Thanks.
---
If your project is set up for it,
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-215160693
Thanks!
---
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 feature
ena
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12593
---
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 feature
enabled and wishes so, or if the feature is ena
Github user dbtsai commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214923459
LGTM. Merged into master. Thanks.
---
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 no
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214873998
Merged build finished. Test PASSed.
---
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214874003
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214873844
**[Test build #57020 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57020/consoleFull)**
for PR 12593 at commit
[`3e5f58b`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214863178
**[Test build #57020 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57020/consoleFull)**
for PR 12593 at commit
[`3e5f58b`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214859907
Merged build finished. Test FAILed.
---
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214859909
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214859863
**[Test build #57017 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57017/consoleFull)**
for PR 12593 at commit
[`d9901f7`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214851129
**[Test build #57017 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57017/consoleFull)**
for PR 12593 at commit
[`d9901f7`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214560608
**[Test build #56940 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56940/consoleFull)**
for PR 12593 at commit
[`50e5d1c`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214560616
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214560612
Merged build finished. Test FAILed.
---
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214560392
**[Test build #56940 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56940/consoleFull)**
for PR 12593 at commit
[`50e5d1c`](https://gi
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214559694
Updated, thanks! I'll do the Since annotation later
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. I
Github user dbtsai commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214543499
Do you want to add the test code for both `MultivariateGaussian.scala` and
`Utils.scala`?
---
If your project is set up for it, you can reply to this email and have you
Github user dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/12593#discussion_r60996366
--- Diff:
mllib-local/src/main/scala/org/apache/spark/ml/stat/distribution/MultivariateGaussian.scala
---
@@ -0,0 +1,131 @@
+/*
+ * Licensed to the A
Github user dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/12593#discussion_r60996174
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/clustering/GaussianMixture.scala ---
@@ -17,17 +17,19 @@
package org.apache.spark.ml.clustering
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214494498
Merged build finished. Test PASSed.
---
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214494500
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214494351
**[Test build #56912 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56912/consoleFull)**
for PR 12593 at commit
[`98f398c`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214478040
**[Test build #56912 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56912/consoleFull)**
for PR 12593 at commit
[`98f398c`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214471645
Merged build finished. Test FAILed.
---
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214471646
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214471592
**[Test build #56906 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56906/consoleFull)**
for PR 12593 at commit
[`3d310f2`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214464159
**[Test build #56906 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56906/consoleFull)**
for PR 12593 at commit
[`3d310f2`](https://gi
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/12593#issuecomment-214463944
CC: @dbtsai Making you aware of this since it touches mllib-local
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
30 matches
Mail list logo
