Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/18538
@jkbradley @mgaido91 I just sent #19648 to move test data to data/mllib,
please feel free to review it. Thanks.
---
-
To unsu
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/18538
@mgaido91 Don't worry, I'll post a follow-up PR for discussion in a few
days. Thanks.
---
-
To unsubscribe, e-mail: reviews-
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/18538
@jkbradley I am not sure that we should put the data for tests of the ml
package in the mllib package. Is this the right approach?
---
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/18538
@yanboliang @mgaido91 I just saw this PR. It creates a new test data
directory. Could you please send a quite update to move the data to the
existing data directory: https://github.com/apache/s
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/18538
@yanboliang yes, thank you very much.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/18538
@mgaido91 I opened
[SPARK-21981](https://issues.apache.org/jira/browse/SPARK-21981) for Python
API, would you like to work on it? Thanks.
---
--
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/18538
I'm merging this into master, thanks for all. If anyone has more comments,
we can address them in follow-up PRs.
---
-
To uns
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/18538
@yanboliang I addressed them. Thank you very much for your time, help and
your great reviews.
---
-
To unsubscribe, e-mail: rev
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81666/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18538
**[Test build #81666 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81666/testReport)**
for PR 18538 at commit
[`a7c1481`](https://github.com/apache/spark/commit/a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18538
**[Test build #81666 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81666/testReport)**
for PR 18538 at commit
[`a7c1481`](https://github.com/apache/spark/commit/a7
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/18538
@mgaido91 These are my last comments, it should be ready to merge once they
are addressed. Thanks for your contribution.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81639/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18538
**[Test build #81639 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81639/testReport)**
for PR 18538 at commit
[`b0b7853`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18538
**[Test build #81639 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81639/testReport)**
for PR 18538 at commit
[`b0b7853`](https://github.com/apache/spark/commit/b0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81463/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18538
**[Test build #81463 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81463/testReport)**
for PR 18538 at commit
[`7b8149a`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18538
**[Test build #81463 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81463/testReport)**
for PR 18538 at commit
[`7b8149a`](https://github.com/apache/spark/commit/7b
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/18538
@mgaido91 I left some minor comments, otherwise, this looks good. Thanks.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@sp
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81369/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
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 project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18538
**[Test build #81369 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81369/testReport)**
for PR 18538 at commit
[`9abe9e5`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18538
**[Test build #81369 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81369/testReport)**
for PR 18538 at commit
[`9abe9e5`](https://github.com/apache/spark/commit/9a
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/18538
Jenkins, test this please.
---
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
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/18538
the build error is caused by some file being present unexpectedly for
SparkR. This issue is unrelated with the PR (which doesn't even affect SparkR).
I am not sure whether someone is working on the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81316/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
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 project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18538
**[Test build #81316 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81316/testReport)**
for PR 18538 at commit
[`9abe9e5`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18538
**[Test build #81316 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81316/testReport)**
for PR 18538 at commit
[`9abe9e5`](https://github.com/apache/spark/commit/9a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
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 project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81287/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18538
**[Test build #81287 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81287/testReport)**
for PR 18538 at commit
[`45d1380`](https://github.com/apache/spark/commit/4
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18538
**[Test build #81287 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81287/testReport)**
for PR 18538 at commit
[`45d1380`](https://github.com/apache/spark/commit/45
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80862/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
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 project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18538
**[Test build #80862 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80862/testReport)**
for PR 18538 at commit
[`a7db896`](https://github.com/apache/spark/commit/a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18538
**[Test build #80862 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80862/testReport)**
for PR 18538 at commit
[`a7db896`](https://github.com/apache/spark/commit/a7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80860/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18538
**[Test build #80860 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80860/testReport)**
for PR 18538 at commit
[`a4ca3cd`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
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 project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18538
**[Test build #80860 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80860/testReport)**
for PR 18538 at commit
[`a4ca3cd`](https://github.com/apache/spark/commit/a4
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/18538
@mgaido91 I made another pass and left some comments, mainly about naming
and annotation. This looks in good shape now. I'd suggest to following the name
in sklearn, which should be easy to under
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/18538
@yanboliang thanks for your review.
I refactored the code according to your suggestions and I removed the
cosine implementation.
Might you please review it now?
Thanks.
---
If your pro
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
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 project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80453/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18538
**[Test build #80453 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80453/testReport)**
for PR 18538 at commit
[`ffc17f9`](https://github.com/apache/spark/commit/f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18538
**[Test build #80453 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80453/testReport)**
for PR 18538 at commit
[`ffc17f9`](https://github.com/apache/spark/commit/ff
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80285/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18538
**[Test build #80285 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80285/testReport)**
for PR 18538 at commit
[`923418a`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
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 project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18538
**[Test build #80285 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80285/testReport)**
for PR 18538 at commit
[`923418a`](https://github.com/apache/spark/commit/92
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
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 project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80281/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18538
**[Test build #80281 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80281/testReport)**
for PR 18538 at commit
[`cfcb106`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18538
**[Test build #80281 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80281/testReport)**
for PR 18538 at commit
[`cfcb106`](https://github.com/apache/spark/commit/cf
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18538
retest this please
---
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
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18538
ok to test
---
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
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18538
test this please
---
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,
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/18538
@gatorsmile Could you help to trigger the test job? It seems I can't do it
now. Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/18538
ok to test
---
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
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/18538
Jenkins, test this please.
---
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 w
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18538
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 feat
65 matches
Mail list logo
