Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/14784
merged this to master and branch-2.0
---
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
en
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/14784
LGTM. Thanks for finding and fixing this problem.
---
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 th
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14784
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/14784
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65626/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14784
**[Test build #65626 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65626/consoleFull)**
for PR 14784 at commit
[`c91d02a`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14784
**[Test build #65626 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65626/consoleFull)**
for PR 14784 at commit
[`c91d02a`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14784
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/14784
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65589/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14784
**[Test build #65589 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65589/consoleFull)**
for PR 14784 at commit
[`e635164`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14784
**[Test build #65589 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65589/consoleFull)**
for PR 14784 at commit
[`e635164`](https://github.com/apache/spark/commit/e
Github user zjffdu commented on the issue:
https://github.com/apache/spark/pull/14784
@shivaram @felixcheung Sorry for late response, I just rebase the PR and
also take spark.master over master. Please help review.
---
If your project is set up for it, you can reply to this email a
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/14784
@shivaram @zjffdu my preference would be to keep the existing precedence
order, from our discussion above.
If we think we should change it at this point, then we should be handling
ever
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14784
@zjffdu Sorry for the delay. I think the change looks pretty good. We could
just add a `logWarning` in case we find a collision ? Also could you bring this
up to date with the master branch ?
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14784
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64566/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14784
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14784
**[Test build #64566 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64566/consoleFull)**
for PR 14784 at commit
[`986cddc`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14784
**[Test build #64566 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64566/consoleFull)**
for PR 14784 at commit
[`986cddc`](https://github.com/apache/spark/commit/9
Github user zjffdu commented on the issue:
https://github.com/apache/spark/pull/14784
@shivaram @felixcheung Could you help review it ? 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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14784
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/14784
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64340/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14784
**[Test build #64340 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64340/consoleFull)**
for PR 14784 at commit
[`635d98d`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14784
**[Test build #64340 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64340/consoleFull)**
for PR 14784 at commit
[`635d98d`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14784
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64339/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14784
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14784
**[Test build #64339 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64339/consoleFull)**
for PR 14784 at commit
[`20c36c4`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14784
**[Test build #64339 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64339/consoleFull)**
for PR 14784 at commit
[`20c36c4`](https://github.com/apache/spark/commit/2
26 matches
Mail list logo
