Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/14698
LGTM - merging to master/2.0. Thanks for working on 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 project does not
Github user lw-lin commented on the issue:
https://github.com/apache/spark/pull/14698
Thanks @hvanhovell for the review!
This patch has been updated.
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14698
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64395/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14698
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/14698
**[Test build #64395 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64395/consoleFull)**
for PR 14698 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14698
**[Test build #64395 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64395/consoleFull)**
for PR 14698 at commit
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/14698
@lw-lin this looks pretty good. I have left a few comments.
---
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 JoshRosen commented on the issue:
https://github.com/apache/spark/pull/14698
@davies, could you take a look at this one?
---
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/14698
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63979/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14698
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/14698
**[Test build #63979 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63979/consoleFull)**
for PR 14698 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14698
**[Test build #63979 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63979/consoleFull)**
for PR 14698 at commit
12 matches
Mail list logo
