Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/13194
Since this is very straight forward, I'll merge it into master. Any
comments are still appreciated, and I can address them in follow up work.
Thanks for review! @holdenk @BryanCutler
---
If yo
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/13194
ping @MLnick & @davies for final review if they have bandwidth around this?
---
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 pr
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13194
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/13194
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66827/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13194
**[Test build #66827 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66827/consoleFull)**
for PR 13194 at commit
[`cc85a92`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13194
**[Test build #66827 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66827/consoleFull)**
for PR 13194 at commit
[`cc85a92`](https://github.com/apache/spark/commit/c
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/13194
Just one question, otherwise LGTM
---
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
enabl
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/13194
Resolved merge conflicts, ping @holdenk @MLnick @srowen @jkbradley to take
a look when you available. This is just add Python API and should be very
straight forward to move ahead. Thanks.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13194
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/5/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13194
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/13194
**[Test build #5 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/5/consoleFull)**
for PR 13194 at commit
[`8764e83`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13194
**[Test build #5 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/5/consoleFull)**
for PR 13194 at commit
[`8764e83`](https://github.com/apache/spark/commit/8
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/13194
This seems pretty reasonable, do you have the time to update it to master
and then maybe we could get @MLnick to take a quick look?
---
If your project is set up for it, you can reply to this email
13 matches
Mail list logo