Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/15119
LGTM. Merging 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 feature
enabled and wishes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15119
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/15119
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71173/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15119
**[Test build #71173 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71173/testReport)**
for PR 15119 at commit
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/15119
Thanks @vanzin , good points. I took another read through the doc and
thought it was better to move things around a little. I also noticed a
misplaced tag in another setting. Let me know if
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15119
**[Test build #71173 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71173/testReport)**
for PR 15119 at commit
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/15119
Hi @vanzin, any more comments or think this could be merged?
---
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/15119
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/15119
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71100/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15119
**[Test build #71100 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71100/testReport)**
for PR 15119 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15119
**[Test build #71100 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71100/testReport)**
for PR 15119 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15119
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71001/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15119
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/15119
**[Test build #71001 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71001/testReport)**
for PR 15119 at commit
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/15119
@vanzin and @themodernlife , I have this fixed up to use the additional
repositories when loading ivy settings from a file. Added a new test for
loading a settings and fixed up the docs for
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15119
**[Test build #71001 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71001/testReport)**
for PR 15119 at commit
Github user themodernlife commented on the issue:
https://github.com/apache/spark/pull/15119
@BryanCutler
- #1 I think you're right... the naming of that key is unfortuante,
`spark.jars.ivyUserDir` or something would have been better... it affects
`defaultIvyUserDir`
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/15119
Not sure about (1), maybe @brkyvz knows since I think he wrote that code.
(2) sounds ok to me.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/15119
Sorry for the delay @vanzin and @themodernlife , I was only able to start
looking at this today. I have a couple questions for you guys
1. I think the docs might not be quite right
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/15119
Thanks for taking a look @vanzin, it'll be a few days before I can work on
this but I'll ping you and @themodernlife when ready.
---
If your project is set up for it, you can reply to this
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/15119
Hi @themodernlife, thanks for testing this out! Glad it worked for you,
I'll try to ping some others and see if I can get more interest. ping
@JoshRosen @brkyvz
---
If your project is set
Github user themodernlife commented on the issue:
https://github.com/apache/spark/pull/15119
FYI I tried this out in our environment
- Firewall (no access to maven central)
- Custom ivysettings.xml to point to our internal Artifactory
Everything worked just as I'd
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15119
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/15119
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65506/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15119
**[Test build #65506 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65506/consoleFull)**
for PR 15119 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15119
**[Test build #65506 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65506/consoleFull)**
for PR 15119 at commit
26 matches
Mail list logo