Github user mpjlu commented on the issue:
https://github.com/apache/spark/pull/18624
Because I don't have the environment to continue this work, I will close
it. Thanks.
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18624
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84752/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18624
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18624
**[Test build #84752 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84752/testReport)**
for PR 18624 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18624
**[Test build #84752 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84752/testReport)**
for PR 18624 at commit
Github user mpjlu commented on the issue:
https://github.com/apache/spark/pull/18624
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18624
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18624
**[Test build #84750 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84750/testReport)**
for PR 18624 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18624
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84750/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18624
**[Test build #84750 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84750/testReport)**
for PR 18624 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18624
**[Test build #84749 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84749/testReport)**
for PR 18624 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18624
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18624
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84749/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18624
**[Test build #84749 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84749/testReport)**
for PR 18624 at commit
Github user mpjlu commented on the issue:
https://github.com/apache/spark/pull/18624
Hi @holdenk, this is the PR we have discussed in Strata conference.
![matrix
Github user mpjlu commented on the issue:
https://github.com/apache/spark/pull/18624
Thanks @WeichenXu123 , I will think about the method to simplify the code.
---
-
To unsubscribe, e-mail:
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/18624
But, I agree the issue @MLnick mentioned, the code now looks convoluted,
can you try to simplify it ?
---
-
To
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/18624
ping @WeichenXu123
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user mpjlu commented on the issue:
https://github.com/apache/spark/pull/18624
Thanks @MLnick , I think the ML ALS suite is ok, just MLLIB ALS suite is
too simple. One possible enhancement is to add the same test cases as ML ALS
suite. How do you think about it?
---
If your
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/18624
@mpjlu also feel free to look into enhancing the tests. One issue however
is we don't want to add to the run time too much as the ALS suite already is
very heavy on time.
---
If your project is
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/18624
@mpjlu sorry for delay on this. I think the idea of the change is good but
I still need to review in detail. One concern I have is it now looks quite
convoluted, so I want to see if we can simplify
Github user mpjlu commented on the issue:
https://github.com/apache/spark/pull/18624
Hi @srowen @MLnick @jkbradley @mengxr @yanboliang
Is this change acceptable? if it is acceptable, I will update ALS ML code
following this method. Also update Test Suite, which are too simple,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18624
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/18624
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79664/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18624
**[Test build #79664 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79664/testReport)**
for PR 18624 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18624
**[Test build #79664 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79664/testReport)**
for PR 18624 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18624
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79655/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18624
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/18624
**[Test build #79655 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79655/testReport)**
for PR 18624 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18624
**[Test build #79655 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79655/testReport)**
for PR 18624 at commit
Github user mpjlu commented on the issue:
https://github.com/apache/spark/pull/18624
I have checked the results with the master method, the recommendation
results are right.
The master TestSuite is too simple, should be updated. I will update it.
Thanks.
---
If your
Github user mpjlu commented on the issue:
https://github.com/apache/spark/pull/18624
If no poll, we have to use toArray.sorted, which performance is bad.
---
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
Github user mpjlu commented on the issue:
https://github.com/apache/spark/pull/18624
We need the value is in order here.
---
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 mpjlu commented on the issue:
https://github.com/apache/spark/pull/18624
An user block, after Cartesian, will generate many blocks(Number of Item
blocks), all these blocks should be aggregated. Thanks.
---
If your project is set up for it, you can reply to this email
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18624
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/18624
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79597/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18624
**[Test build #79597 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79597/testReport)**
for PR 18624 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18624
**[Test build #79597 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79597/testReport)**
for PR 18624 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18624
**[Test build #79587 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79587/testReport)**
for PR 18624 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18624
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/18624
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79587/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18624
**[Test build #79587 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79587/testReport)**
for PR 18624 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18624
**[Test build #79584 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79584/testReport)**
for PR 18624 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18624
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79584/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18624
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/18624
**[Test build #79584 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79584/testReport)**
for PR 18624 at commit
46 matches
Mail list logo