Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14597
Nice one. I like this change. Merged to master.
---
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 user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14597
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/14597
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65658/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14597
**[Test build #65658 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65658/consoleFull)**
for PR 14597 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14597
**[Test build #65658 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65658/consoleFull)**
for PR 14597 at commit
Github user mpjlu commented on the issue:
https://github.com/apache/spark/pull/14597
No problem. thanks very much @srowen .
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/14597
This is ready to go ... except that it needs a rebase now, sorry.
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14597
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65585/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14597
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/14597
**[Test build #65585 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65585/consoleFull)**
for PR 14597 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14597
**[Test build #65585 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65585/consoleFull)**
for PR 14597 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14597
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65563/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14597
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/14597
**[Test build #65563 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65563/consoleFull)**
for PR 14597 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14597
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65559/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14597
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/14597
**[Test build #65559 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65559/consoleFull)**
for PR 14597 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14597
**[Test build #65563 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65563/consoleFull)**
for PR 14597 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14597
**[Test build #65559 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65559/consoleFull)**
for PR 14597 at commit
Github user mpjlu commented on the issue:
https://github.com/apache/spark/pull/14597
Thanks very much, I am in holiday now, will update the code this Sunday.
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14597
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65359/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14597
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/14597
**[Test build #65359 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65359/consoleFull)**
for PR 14597 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14597
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
enabled
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14597
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65357/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14597
**[Test build #65357 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65357/consoleFull)**
for PR 14597 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14597
**[Test build #65359 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65359/consoleFull)**
for PR 14597 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14597
**[Test build #65357 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65357/consoleFull)**
for PR 14597 at commit
Github user mpjlu commented on the issue:
https://github.com/apache/spark/pull/14597
hi @srowen , I have updated the code for some MiMa test error. Could you
please review it again. Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14597
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65318/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14597
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/14597
**[Test build #65318 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65318/consoleFull)**
for PR 14597 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14597
**[Test build #65318 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65318/consoleFull)**
for PR 14597 at commit
Github user mpjlu commented on the issue:
https://github.com/apache/spark/pull/14597
hi @srowen , we have moved isSorted in ChiSqSelectorModel. There is error
message "*** method isSorted(Array[Int])Boolean in class
org.apache.spark.mllib.feature.ChiSqSelectorModel does not have a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14597
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/14597
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65315/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14597
**[Test build #65315 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65315/consoleFull)**
for PR 14597 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14597
**[Test build #65315 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65315/consoleFull)**
for PR 14597 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14597
**[Test build #3259 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3259/consoleFull)**
for PR 14597 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14597
**[Test build #3259 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3259/consoleFull)**
for PR 14597 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14597
**[Test build #65314 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65314/consoleFull)**
for PR 14597 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14597
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65314/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14597
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/14597
**[Test build #65314 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65314/consoleFull)**
for PR 14597 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14597
Jenkins add to whitelist
---
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
Github user mpjlu commented on the issue:
https://github.com/apache/spark/pull/14597
Hi @srowen , Python style fail is updated. 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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14597
**[Test build #3257 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3257/consoleFull)**
for PR 14597 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14597
**[Test build #3257 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3257/consoleFull)**
for PR 14597 at commit
Github user mpjlu commented on the issue:
https://github.com/apache/spark/pull/14597
hi @srowen , thanks.
This is my first PR. learn much from you. thanks very much.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14597
@mpjlu I'm still following this one, and I looked again today. I couldn't
see anything else to change. For a minute I thought we need to update
ChiSqSelectorModelReader / ChiSqSelectorModelWriter
Github user mpjlu commented on the issue:
https://github.com/apache/spark/pull/14597
Hi @yanboliang , could you please kindly review the python code of this PR.
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
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14597
I think this is OK as far as I know though I'd appreciate it if any Python
people can look at the Python side.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user mpjlu commented on the issue:
https://github.com/apache/spark/pull/14597
Hi @srowen , I have added Python API and test cases for ChiSqSelector.
Could you kindly review it again. Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user mpjlu commented on the issue:
https://github.com/apache/spark/pull/14597
Sure, I can update the Python API.
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/14597
OK, that looks solid for Scala. I think we need to update the Python API
too. That much should be straightforward. Are you in a position to do that -- I
think we can call in advice from people that
Github user mpjlu commented on the issue:
https://github.com/apache/spark/pull/14597
Hi @srowen , I will update the Python API to match this changes. Now, the
current Python API is not conflict with the changes.
---
If your project is set up for it, you can reply to this email
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14597
I think this will require a little update to the Python API to match. Not
sure about SparkR
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
57 matches
Mail list logo
