Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/15981
LGTM. The build for the latest commit passed. Merging to master and 2.1.
---
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/15981
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/15981
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69273/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15981
**[Test build #69273 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69273/consoleFull)**
for PR 15981 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15981
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69277/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15981
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/15981
**[Test build #69277 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69277/consoleFull)**
for PR 15981 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15981
**[Test build #69277 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69277/consoleFull)**
for PR 15981 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/15981
Looks like I removed the warning from SparkEnv at some point, let me add it
back.
---
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 zsxwing commented on the issue:
https://github.com/apache/spark/pull/15981
> I think it's fine to just have a warning here.
I cannot find the warning log in your latest commit. Otherwise, LGTM.
---
If your project is set up for it, you can reply to this email and
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15981
**[Test build #69273 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69273/consoleFull)**
for PR 15981 at commit
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/15981
> I think it's fine to just have a warning here.
Sounds good. Just make sure you mention it when updating the doc.
---
If your project is set up for it, you can reply to this email and
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/15981
> Since it sends the key over the wire, I think it's better to throw an
exception if securityManager.isSaslEncryptionEnabled() is false.
We don't do that now, and there's a whole lot of
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/15981
Ping.
---
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 so, or if the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15981
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/15981
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69033/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15981
**[Test build #69033 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69033/consoleFull)**
for PR 15981 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15981
**[Test build #69033 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69033/consoleFull)**
for PR 15981 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15981
**[Test build #69029 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69029/consoleFull)**
for PR 15981 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15981
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/15981
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69029/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15981
**[Test build #69029 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69029/consoleFull)**
for PR 15981 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/15981
/cc @zsxwing @JoshRosen
---
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
23 matches
Mail list logo