Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/7657
---
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 feature is enab
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-130473902
Merged to master and 1.5. 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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-130473709
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
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-130473640
42nd time is the charm, I guess. Merging.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-130473434
[Test build #40661 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40661/console)
for PR 7657 at commit
[`7f3e1ff`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-130439402
[Test build #40661 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40661/consoleFull)
for PR 7657 at commit
[`7f3e1ff`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-130438932
Merged build started.
---
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 pull request:
https://github.com/apache/spark/pull/7657#issuecomment-130438911
Merged build triggered.
---
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 h
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-130435469
> I won't have a chance to test this out on a real cluster
I tested yesterday, client and cluster mode, conf and command line options.
All seems fine.
---
If yo
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/7657#discussion_r36907667
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -528,7 +528,11 @@ class SparkContext(config: SparkConf) extends Logging
with Execu
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-130432155
Looks fine, though I won't have a chance to test this out on a real cluster
and I think we should before the release. Feel free to merge it.
---
If your project is s
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-130368135
Alright, will merge later today.
---
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 vanzin commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-130098313
Tested this locally and it looks good. @andrewor14 did you have any other
comments?
Test failures are unrelated (pyspark?) so I'll just ignore them.
---
If your
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-130072036
[Test build #40489 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40489/console)
for PR 7657 at commit
[`6da06c4`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-130072414
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-130002773
[Test build #40489 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40489/consoleFull)
for PR 7657 at commit
[`6da06c4`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-130001677
Merged build started.
---
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 pull request:
https://github.com/apache/spark/pull/7657#issuecomment-130001656
Merged build triggered.
---
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 h
Github user neurons commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-130001489
Thank you @vanzin and @andrewor14 for your reviews.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If yo
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-13800
Alright, LGTM. Let's try jenkins again. retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-13846
(I'll try to run some local tests later on too.)
---
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 p
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-129707066
[Test build #40376 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40376/console)
for PR 7657 at commit
[`6da06c4`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-129707136
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-129680987
[Test build #40376 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40376/consoleFull)
for PR 7657 at commit
[`6da06c4`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-129680198
Merged build started.
---
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 pull request:
https://github.com/apache/spark/pull/7657#issuecomment-129680139
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-129632505
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-129632441
[Test build #40312 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40312/console)
for PR 7657 at commit
[`682626e`](https://github.
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/7657#discussion_r36688007
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -86,7 +86,7 @@ private[yarn] class YarnAllocator(
private var exec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-129590684
[Test build #40312 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40312/consoleFull)
for PR 7657 at commit
[`682626e`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-129590307
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-129590380
Merged build started.
---
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 pull request:
https://github.com/apache/spark/pull/7657#issuecomment-129583234
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-129583153
[Test build #40298 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40298/console)
for PR 7657 at commit
[`d54e732`](https://github.
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/7657#discussion_r36675822
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -86,7 +86,7 @@ private[yarn] class YarnAllocator(
private var exec
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/7657#discussion_r36674848
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -86,7 +86,7 @@ private[yarn] class YarnAllocator(
private var exec
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/7657#discussion_r36674567
--- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala ---
@@ -960,6 +959,10 @@ object Client extends Logging {
val sparkConf = new Sp
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/7657#discussion_r36674548
--- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala ---
@@ -960,6 +959,10 @@ object Client extends Logging {
val sparkConf = new Sp
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/7657#discussion_r36674458
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -2286,6 +2286,17 @@ private[spark] object Utils extends Logging {
isInDirectory(
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/7657#discussion_r36674387
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -528,7 +528,11 @@ class SparkContext(config: SparkConf) extends Logging
with ExecutorA
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/7657#discussion_r36674280
--- Diff: core/src/main/scala/org/apache/spark/SparkConf.scala ---
@@ -476,6 +477,24 @@ class SparkConf(loadDefaults: Boolean) extends
Cloneable with Logging {
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-129536263
[Test build #40298 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40298/consoleFull)
for PR 7657 at commit
[`d54e732`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-129532451
Merged build started.
---
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 pull request:
https://github.com/apache/spark/pull/7657#issuecomment-129532420
Merged build triggered.
---
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 h
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/7657#discussion_r36581032
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -2286,6 +2286,17 @@ private[spark] object Utils extends Logging {
isInDirectory(
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/7657#discussion_r36487340
--- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala ---
@@ -958,6 +957,8 @@ object Client extends Logging {
val sparkConf = new Spa
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/7657#discussion_r36480290
--- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala ---
@@ -958,6 +957,8 @@ object Client extends Logging {
val sparkConf = new
Github user neurons commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128541535
@andrewor14 -- addressed the changes you requested. any further thoughts?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128521398
[Test build #40069 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40069/console)
for PR 7657 at commit
[`8ceeaa6`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128521485
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128476355
[Test build #40069 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40069/consoleFull)
for PR 7657 at commit
[`8ceeaa6`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128475075
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128475095
Merged build started.
---
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 vanzin commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128474986
retest this please
---
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 feat
Github user neurons commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128473280
retest this please
---
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 fea
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128466869
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128464082
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128464077
[Test build #257 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SlowSparkPullRequestBuilder/257/console)
for PR 7657 at commit
[`8ceeaa6`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128462971
Merged build started.
---
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 pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128462953
Merged build triggered.
---
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 h
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128462849
[Test build #257 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SlowSparkPullRequestBuilder/257/consoleFull)
for PR 7657 at commit
[`8ceeaa6`](https://gith
Github user squito commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128462817
Jenkins, add to whitelist
---
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 squito commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128462332
Jenkins retest this please
---
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 t
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128462566
Merged build started.
---
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 pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128462494
Merged build triggered.
---
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 h
Github user neurons commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128461559
retest this please
---
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 fea
Github user squito commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128459876
Jenkins, ok to test
---
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 fea
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128458686
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128458679
[Test build #40053 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40053/console)
for PR 7657 at commit
[`8ceeaa6`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128457696
[Test build #40053 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40053/consoleFull)
for PR 7657 at commit
[`8ceeaa6`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128456471
Merged build started.
---
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 pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128456445
Merged build triggered.
---
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 h
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/7657#discussion_r36375900
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -2286,6 +2286,17 @@ private[spark] object Utils extends Logging {
isInDirect
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/7657#discussion_r36375889
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -2286,6 +2286,17 @@ private[spark] object Utils extends Logging {
isInDirect
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/7657#discussion_r36375697
--- Diff: docs/running-on-yarn.md ---
@@ -199,7 +199,7 @@ If you need a reference to the proper location to put
log files in the YARN so t
spark.exec
Github user neurons commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128172923
@andrewor14 @srowen, made the changes you've requested.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128172866
LGTM after fixing small remaining nit. Will let @andrewor14 take another
look.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/7657#discussion_r36363711
--- Diff: core/src/main/scala/org/apache/spark/SparkConf.scala ---
@@ -469,6 +470,24 @@ class SparkConf(loadDefaults: Boolean) extends
Cloneable with Logging {
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128162919
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128162812
[Test build #236 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SlowSparkPullRequestBuilder/236/console)
for PR 7657 at commit
[`95152ee`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128156040
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128155991
[Test build #39902 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39902/console)
for PR 7657 at commit
[`95152ee`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128116156
[Test build #39902 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39902/consoleFull)
for PR 7657 at commit
[`95152ee`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128115926
Merged build started.
---
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 pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128115889
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128115331
Merged build started.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128115437
[Test build #236 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SlowSparkPullRequestBuilder/236/consoleFull)
for PR 7657 at commit
[`95152ee`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128115267
Merged build triggered.
---
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 h
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128114141
retest this please
---
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 feat
Github user neurons commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128113297
retest this please
---
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 fea
Github user neurons commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128102979
Jenkins, retest this please
---
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 pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128097323
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128092952
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-128093013
Merged build started.
---
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 pull request:
https://github.com/apache/spark/pull/7657#issuecomment-127831495
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-127831300
[Test build #39792 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39792/console)
for PR 7657 at commit
[`dafd327`](https://github.
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/7657#issuecomment-127816129
Just one thing to fix in SparkConf and I think this is good to go.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/7657#discussion_r36262926
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -86,7 +86,7 @@ private[yarn] class YarnAllocator(
private var exec
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/7657#discussion_r36262836
--- Diff: core/src/test/scala/org/apache/spark/SparkContextSuite.scala ---
@@ -285,4 +285,12 @@ class SparkContextSuite extends SparkFunSuite with
LocalSparkCo
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/7657#discussion_r36262812
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/ApplicationMaster.scala ---
@@ -64,7 +64,8 @@ private[spark] class ApplicationMaster(
/
1 - 100 of 153 matches
Mail list logo
