Github user markgrover commented on a diff in the pull request:
https://github.com/apache/spark/pull/7739#discussion_r36826550
--- Diff: core/src/main/scala/org/apache/spark/SparkConf.scala ---
@@ -479,6 +484,23 @@ class SparkConf(loadDefaults: Boolean) extends
Cloneable with Loggi
Github user ggupta81 commented on a diff in the pull request:
https://github.com/apache/spark/pull/8113#discussion_r36826491
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregates.scala
---
@@ -630,59 +632,114 @@ case class CombineSetsAndSumFunc
Github user maropu closed the pull request at:
https://github.com/apache/spark/pull/8099
---
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 markgrover commented on the pull request:
https://github.com/apache/spark/pull/8093#issuecomment-130159464
Thanks for the review, @squito. I did consider that as well. The UI listens
to whatever is sent as `SparkListenerTaskEnd` message by the
DAGScheduler::handleTaskCompl
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/8042#issuecomment-130159360
@feynmanliang The contract for Java's `Object` is that `a.equals(b)`
implies `a.hashCode == b.hashCode`. So usually we need to implement both. The
problem with `hashCode`
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7878#issuecomment-130157632
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/8068#issuecomment-130156962
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/8068#issuecomment-130156894
[Test build #40570 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40570/console)
for PR 8068 at commit
[`ff698fa`](https://github.
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8113#discussion_r36825930
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregates.scala
---
@@ -630,59 +632,114 @@ case class CombineSetsAndSumFunctio
Github user markgrover commented on a diff in the pull request:
https://github.com/apache/spark/pull/8093#discussion_r36825910
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -435,19 +437,24 @@ private[yarn] class YarnAllocator(
//
Github user markgrover commented on a diff in the pull request:
https://github.com/apache/spark/pull/8093#discussion_r36825911
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -423,6 +424,7 @@ private[yarn] class YarnAllocator(
for (com
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7878#issuecomment-130156154
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/7878#issuecomment-130156164
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/8085
---
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 mengxr commented on the pull request:
https://github.com/apache/spark/pull/8085#issuecomment-130156074
Merged into master and branch-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
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/7878#issuecomment-130156023
Jenkins, test 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 mengxr commented on the pull request:
https://github.com/apache/spark/pull/8087#issuecomment-130155885
What if `condition` requires at least one batch to work correctly? This is
not the case for streaming ML algorithms, but I'm not sure for other streaming
unit tests.
--
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8113#issuecomment-130155854
[Test build #40589 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40589/consoleFull)
for PR 8113 at commit
[`15c9e31`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8113#issuecomment-130155782
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/8113#issuecomment-130155769
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 yhuai commented on the pull request:
https://github.com/apache/spark/pull/8113#issuecomment-130155568
@hvanhovell Alright. I added the `ignoreNulls` flag.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If you
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/7411
---
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 pwendell commented on the pull request:
https://github.com/apache/spark/pull/7411#issuecomment-130155218
Okay will merge this - I've been keeping things in a separate repo and it's
much better to have it in the upstream in case others want to modify it.
---
If your projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8116#issuecomment-130154323
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/4380#issuecomment-130154342
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/8087#issuecomment-130153820
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/8116#issuecomment-130153739
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/4380#issuecomment-130153749
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 yhuai commented on the pull request:
https://github.com/apache/spark/pull/8038#issuecomment-130153744
```
[error]
/home/jenkins/workspace/NewSparkPullRequestBuilder/sql/core/src/test/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregationIteratorSuite.scala:4
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8116#issuecomment-130153735
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/4380#issuecomment-130153737
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 scwf commented on the pull request:
https://github.com/apache/spark/pull/4380#issuecomment-130153626
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 featur
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/4380#issuecomment-130153233
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 scwf commented on the pull request:
https://github.com/apache/spark/pull/4380#issuecomment-130152470
yes, since we upgrade the hive version to 1.2.1, we should adapt the token
tree in hiveql, the old one is not correct in 1.2.1. Updated
---
If your project is set up for
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/4380#issuecomment-130151463
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/4380#issuecomment-130151421
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/8038#issuecomment-130150506
[Test build #1470 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1470/console)
for PR 8038 at commit
[`d4dc9ca`](https://github
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8120#issuecomment-130150558
[Test build #1472 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1472/consoleFull)
for PR 8120 at commit
[`615d08e`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8120#issuecomment-130150533
[Test build #1471 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1471/consoleFull)
for PR 8120 at commit
[`615d08e`](https://git
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8084
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7957#issuecomment-130150454
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/7957#issuecomment-130150422
[Test build #40581 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40581/console)
for PR 7957 at commit
[`4cd4fad`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8087#issuecomment-130150414
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/8087#issuecomment-130150407
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 jkbradley commented on the pull request:
https://github.com/apache/spark/pull/8087#issuecomment-130150269
Jenkins test 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 JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/8084#issuecomment-130150255
LGTM, so I'm going to merge this to master and branch-1.5.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as we
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8120#issuecomment-130149388
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/8120#issuecomment-130149410
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/8120#issuecomment-130149352
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/8087#issuecomment-130149344
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 JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/8120#issuecomment-130149300
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8087#issuecomment-130148853
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/8087#issuecomment-130148813
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/8069#issuecomment-130148732
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 ggupta81 commented on the pull request:
https://github.com/apache/spark/pull/7928#issuecomment-130148639
correct. I will make the changes you have suggested and update the pull
request.
On Wed, Aug 12, 2015 at 8:38 AM, Yin Huai wrote:
> @ggupta81
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8111#issuecomment-130146649
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/8120#issuecomment-130145766
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/8084#issuecomment-130145729
[Test build #1466 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1466/console)
for PR 8084 at commit
[`026f628`](https://github
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8120#issuecomment-130145288
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/8120#issuecomment-130145274
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/7957#issuecomment-130145006
[Test build #40581 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40581/consoleFull)
for PR 7957 at commit
[`4cd4fad`](https://gith
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/8120#issuecomment-130144784
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8120#issuecomment-130144544
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/8084#issuecomment-130144657
[Test build #40580 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40580/consoleFull)
for PR 8084 at commit
[`026f628`](https://gith
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/8023#discussion_r36823588
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/sources/interfaces.scala ---
@@ -565,6 +565,7 @@ abstract class HadoopFsRelation
private[sql](m
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7957#issuecomment-13010
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/7957#issuecomment-130144415
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 chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/8023#discussion_r36823532
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/sources/interfaces.scala ---
@@ -565,6 +565,7 @@ abstract class HadoopFsRelation
private[sql](m
Github user rekhajoshm commented on the pull request:
https://github.com/apache/spark/pull/7602#issuecomment-130144282
Thanks @srowen @andrewor14 updated push with tested UISelenium version
which consistently passes (tested more than 20 times). I was curious to know
why concise versi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7602#issuecomment-130144078
[Test build #40579 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40579/consoleFull)
for PR 7602 at commit
[`eee1733`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8087#issuecomment-130143998
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/8087#issuecomment-130143989
[Test build #40578 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40578/console)
for PR 8087 at commit
[`48f43c8`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8087#issuecomment-130143930
[Test build #40578 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40578/consoleFull)
for PR 8087 at commit
[`48f43c8`](https://gith
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/7957#issuecomment-130143971
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
Github user MechCoder commented on the pull request:
https://github.com/apache/spark/pull/8110#issuecomment-130143654
@mengxr tests pass.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8084#issuecomment-130143114
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/8087#issuecomment-130143144
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 yhuai commented on the pull request:
https://github.com/apache/spark/pull/7928#issuecomment-130143124
@ggupta81 We are having a new implementation of aggregate functions in
1.5.0. The last function having problem in your case is our old implementation.
Let's fix it and I w
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7602#issuecomment-130143174
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/8120#issuecomment-130143082
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/8120#issuecomment-130143141
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/8084#issuecomment-130143175
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/8087#issuecomment-130143106
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/7602#issuecomment-130143123
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 yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/7928#discussion_r36823318
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregates.scala
---
@@ -746,13 +746,11 @@ case class Last(child: Expression) e
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/8053#issuecomment-130142600
This looks like it's going in the right direction. In your previous patch,
you mentioned that `quoteForBatchScript` didn't escape `%` characters; is that
not needed?
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/8120#issuecomment-130142542
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 h
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/8084#issuecomment-130142459
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 h
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/8084#issuecomment-130142470
Wow, this has been the unluckiest PR when it comes to flakiness :(
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8120#issuecomment-130142409
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 vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8053#discussion_r36823225
--- Diff:
yarn/src/main/scala/org/apache/spark/launcher/YarnCommandBuilderUtils.scala ---
@@ -0,0 +1,36 @@
+/*
+ * Licensed to the Apache Software Foun
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8053#discussion_r36823207
--- Diff:
yarn/src/main/scala/org/apache/spark/launcher/YarnCommandBuilderUtils.scala ---
@@ -0,0 +1,36 @@
+/*
+ * Licensed to the Apache Software Foun
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8053#discussion_r36823191
--- Diff:
yarn/src/main/scala/org/apache/spark/launcher/YarnCommandBuilderUtils.scala ---
@@ -0,0 +1,36 @@
+/*
+ * Licensed to the Apache Software Foun
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8053#discussion_r36823172
--- Diff:
yarn/src/main/scala/org/apache/spark/launcher/YarnCommandBuilderUtils.scala ---
@@ -0,0 +1,36 @@
+/*
+ * Licensed to the Apache Software Foun
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7751#issuecomment-130142037
[Test build #40576 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40576/consoleFull)
for PR 7751 at commit
[`d6667da`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8084#issuecomment-130142016
[Test build #1465 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1465/console)
for PR 8084 at commit
[`026f628`](https://github
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8053#discussion_r36823066
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnSparkHadoopUtil.scala ---
@@ -21,6 +21,8 @@ import java.io.File
import java.util.regex.Matc
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7751#issuecomment-130141555
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/7751#issuecomment-130141543
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 JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/8116#issuecomment-130141381
The basic fix here looks good to me, although you might consider using the
TaskContext callback approach to ensure eventual cleanup. We do this inside of
the UnsafeExt
101 - 200 of 1344 matches
Mail list logo