Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/12819
@zhengruifeng Please create JIRAs for the follow-up works:
* Parity check between the ml and mllib test suites, and complement missing
test cases for ml.
* Investigate how to handle
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/12819
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 not have this feature
enabled and
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #66138 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66138/consoleFull)**
for PR 12819 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66138/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #66138 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66138/consoleFull)**
for PR 12819 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66105/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #66105 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66105/consoleFull)**
for PR 12819 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #66105 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66105/consoleFull)**
for PR 12819 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66099/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66100/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #66100 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66100/consoleFull)**
for PR 12819 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #66099 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66099/consoleFull)**
for PR 12819 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #66100 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66100/consoleFull)**
for PR 12819 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #66099 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66099/consoleFull)**
for PR 12819 at commit
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/12819
@zhengruifeng Only left some minor comments, otherwise, looks good. I think
we should also make parity check between the ml and mllib test suites, and
complement missing test cases for ml since
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65869/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #65869 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65869/consoleFull)**
for PR 12819 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #65869 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65869/consoleFull)**
for PR 12819 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65712/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #65712 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65712/consoleFull)**
for PR 12819 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #65712 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65712/consoleFull)**
for PR 12819 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65706/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #65706 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65706/consoleFull)**
for PR 12819 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #65706 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65706/consoleFull)**
for PR 12819 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #65653 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65653/consoleFull)**
for PR 12819 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65653/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #65653 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65653/consoleFull)**
for PR 12819 at commit
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/12819
@sethah It's a good questions, and we have some concerns:
* The Dataset based implementation does not bring much performance
improvement from the test result(even worse in some case).
*
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65599/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #65599 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65599/consoleFull)**
for PR 12819 at commit
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/12819
@yanboliang What went into the decision to use RDD based aggregation? Just
curious, 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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #65599 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65599/consoleFull)**
for PR 12819 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65598/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #65598 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65598/consoleFull)**
for PR 12819 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #65598 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65598/consoleFull)**
for PR 12819 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #65591 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65591/consoleFull)**
for PR 12819 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65591/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #65591 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65591/consoleFull)**
for PR 12819 at commit
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/12819
@zhengruifeng As we discussed offline, we would like to still use RDD
operation underneath rather than UDAF. Please update the PR accordingly. Thanks!
---
If your project is set up for it, you
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/12819
I test on two datasets: a9a and mnist.scale. Download here
http://www.csie.ntu.edu.tw/~cjlin/libsvmtools/datasets/
The test codes are following:
```
import
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65289/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #65289 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65289/consoleFull)**
for PR 12819 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #65289 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65289/consoleFull)**
for PR 12819 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65257/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #65257 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65257/consoleFull)**
for PR 12819 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65259/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #65259 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65259/consoleFull)**
for PR 12819 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #65259 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65259/consoleFull)**
for PR 12819 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #65257 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65257/consoleFull)**
for PR 12819 at commit
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/12819
@yanboliang I will make usage of `weightCol` keeping in line with other
algorithms, and I will do some performance test. Thanks for reviewing this.
---
If your project is set up for it, you
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/12819
@zhengruifeng I saw your implementation switch the training process from
RDD operation to Dataset operation with UDAF. I think we should do some
performance test to verify there is no
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63341/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #63341 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63341/consoleFull)**
for PR 12819 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #63341 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63341/consoleFull)**
for PR 12819 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63338/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #63338 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63338/consoleFull)**
for PR 12819 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #63338 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63338/consoleFull)**
for PR 12819 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63264/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #63264 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63264/consoleFull)**
for PR 12819 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #63264 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63264/consoleFull)**
for PR 12819 at commit
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/12819
I will make a pass soon. 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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63252/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #63252 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63252/consoleFull)**
for PR 12819 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #63252 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63252/consoleFull)**
for PR 12819 at commit
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/12819
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63250/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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
enabled
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #63250 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63250/consoleFull)**
for PR 12819 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #63251 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63251/consoleFull)**
for PR 12819 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63251/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #63251 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63251/consoleFull)**
for PR 12819 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #63250 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63250/consoleFull)**
for PR 12819 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12819
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63165/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #63165 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63165/consoleFull)**
for PR 12819 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12819
**[Test build #63165 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63165/consoleFull)**
for PR 12819 at commit
95 matches
Mail list logo