Github user bijaybisht closed the pull request at:
https://github.com/apache/incubator-spark/pull/568
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. To do so, please top-post your response.
If your project does not have this fe
Github user aarondav commented on the pull request:
https://github.com/apache/incubator-spark/pull/568#issuecomment-35221890
Merged in master and branch-0.9. Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. To do s
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/568#issuecomment-35007472
Merged build finished.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/568#issuecomment-35007475
All automated tests passed.
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/12705/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/568#issuecomment-35004661
Merged build triggered.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/568#issuecomment-35004662
Merged build started.
Github user pwendell commented on the pull request:
https://github.com/apache/incubator-spark/pull/568#issuecomment-35004505
Jenkins, test this please.
Github user bijaybisht commented on the pull request:
https://github.com/apache/incubator-spark/pull/568#issuecomment-34955181
Yes it is getting set via SparkConf now.
Github user pwendell commented on the pull request:
https://github.com/apache/incubator-spark/pull/568#issuecomment-34955056
@aarondav pick this into master as well when you merge it.
Github user pwendell commented on the pull request:
https://github.com/apache/incubator-spark/pull/568#issuecomment-34955035
LGTM
Github user pwendell commented on the pull request:
https://github.com/apache/incubator-spark/pull/568#issuecomment-34955031
@ScrapCodes ah I see - this is changing the system properties iterator to a
spark conf iterator. Makes sense.
Github user ScrapCodes commented on the pull request:
https://github.com/apache/incubator-spark/pull/568#issuecomment-34954162
Not if they have set it via SparkConf. (I guess)
Github user pwendell commented on the pull request:
https://github.com/apache/incubator-spark/pull/568#issuecomment-34953913
@bijaybisht Could you explain somewhere what the bug is that this is
fixing? The jira references "spark.driver.host" but in theory those should have
worked unde
Github user aarondav commented on the pull request:
https://github.com/apache/incubator-spark/pull/568#issuecomment-34905299
Looks good to me, will merge as soon as comments have been addressed.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/568#issuecomment-34577386
Can one of the admins verify this patch?
GitHub user bijaybisht opened a pull request:
https://github.com/apache/incubator-spark/pull/568
fix for https://spark-project.atlassian.net/browse/SPARK-1052
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/apache/incubator-spark
16 matches
Mail list logo
