Github user ankurcha commented on the pull request:
https://github.com/apache/spark/pull/5563#issuecomment-116781619
took care of those @squito
---
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/7086#issuecomment-116783301
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
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/7006#issuecomment-116783952
Hi sorry for slipping I'll look at this 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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7020#issuecomment-116783946
[Test build #35998 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35998/console)
for PR 7020 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7020#issuecomment-116784048
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
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/7036#issuecomment-116785704
but yes, LGTM
---
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 andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/7036#issuecomment-116785573
Is having `*` always strictly faster? If so I wonder if we should just
always append it if it's missing.
---
If your project is set up for it, you can reply to this
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7089#issuecomment-116787969
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7031#issuecomment-116787900
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
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/5423#discussion_r33496190
--- Diff: docs/monitoring.md ---
@@ -256,6 +256,157 @@ still required, though there is only one application
available. Eg. to see the
running
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7031#issuecomment-116787968
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7089#issuecomment-116787888
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
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/7036#issuecomment-116786118
I think the problem is that it's not certain that the FS supports *,
although the usual ones do, it seems.
---
If your project is set up for it, you can reply to this
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7031#issuecomment-116790208
[Test build #36011 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36011/consoleFull)
for PR 7031 at commit
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/7073#discussion_r33496993
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/columnar/ColumnStatsSuite.scala ---
@@ -22,19 +22,19 @@ import
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6921#issuecomment-116792311
[Test build #36012 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36012/consoleFull)
for PR 6921 at commit
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/7055#issuecomment-116792972
@litao-buptsse would you mind opening this against the master branch so
future Spark releases also benefit from the fix?
---
If your project is set up for it, you
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/7060#discussion_r33497360
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameStatSuite.scala ---
@@ -65,22 +67,22 @@ class DataFrameStatSuite extends SparkFunSuite {
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6830#issuecomment-116793699
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6830#issuecomment-116793653
[Test build #36002 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36002/console)
for PR 6830 at commit
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116794010
@cloud-fan This PR looks pretty good to me. Since we're replying on
`StructField.nullable` to skip nullablity check in codegen, that could be kind
of risk, because we
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7090#issuecomment-116795501
[Test build #36013 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36013/consoleFull)
for PR 7090 at commit
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/7069#discussion_r33498497
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/AMDelegationTokenRenewer.scala
---
@@ -65,6 +65,8 @@ private[yarn] class
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7031#issuecomment-116795337
[Test build #36011 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36011/console)
for PR 7031 at commit
Github user steveloughran commented on the pull request:
https://github.com/apache/spark/pull/6935#issuecomment-116795511
(BTW, I'm not going to be looking at this for a couple of weeks; focusing
on a (big) patch
---
If your project is set up for it, you can reply to this email and
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/7069#discussion_r33498576
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkHadoopUtil.scala
---
@@ -334,6 +334,17 @@ class SparkHadoopUtil extends Logging {
*
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/7026
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7094#issuecomment-116814815
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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/6882
---
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
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/7080#discussion_r33505545
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala
---
@@ -534,27 +554,39 @@ private class LogisticCostFun(
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/7080#discussion_r33505552
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/LogisticRegressionSuite.scala
---
@@ -208,8 +208,11 @@ class LogisticRegressionSuite
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/7080#discussion_r33505540
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala
---
@@ -149,15 +158,26 @@ class LogisticRegression(override val
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7091#issuecomment-116828194
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6959#issuecomment-116828193
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6959#issuecomment-116828300
[Test build #36025 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36025/consoleFull)
for PR 6959 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7077#issuecomment-116832856
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7038#issuecomment-116832820
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7048#issuecomment-116832803
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7048#issuecomment-116832857
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7096#issuecomment-116832823
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7077#issuecomment-116832797
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5805#issuecomment-116832906
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5805#issuecomment-116832855
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
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/7048#issuecomment-116831914
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 feature
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/6959#discussion_r33510353
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/UnsafeFixedWidthAggregationMapSuite.scala
---
@@ -33,10 +34,10 @@ class
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7099#issuecomment-116839436
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7099#issuecomment-116840871
[Test build #36036 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36036/consoleFull)
for PR 7099 at commit
Github user rekhajoshm commented on the pull request:
https://github.com/apache/spark/pull/6969#issuecomment-116844189
ack @andrewor14
---
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 andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-116846507
@zsxwing @tdas the hanging issue seems like one that would be good to fix.
Any updates on this patch?
---
If your project is set up for it, you can reply to this
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7089#issuecomment-116846397
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/4399#issuecomment-116846496
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7090#issuecomment-116846495
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
Github user tarekauel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6981#discussion_r33513954
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetime.scala
---
@@ -0,0 +1,255 @@
+/*
+ * Licensed to the
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/6338#issuecomment-116853270
LGTM
---
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
Github user pwendell commented on a diff in the pull request:
https://github.com/apache/spark/pull/7072#discussion_r33516520
--- Diff: core/src/main/scala/org/apache/spark/ui/JettyUtils.scala ---
@@ -166,8 +167,26 @@ private[spark] object JettyUtils extends Logging {
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6338#issuecomment-116853428
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6338#issuecomment-116853253
[Test build #36038 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36038/console)
for PR 6338 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7096#issuecomment-116856329
[Test build #36027 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36027/console)
for PR 7096 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6921#issuecomment-116857401
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
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/6920#issuecomment-116864319
Thanks, merged 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
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/7048#issuecomment-116874802
Hm, not knowing why @SparkQA said the build failed, it was actually a
success. Anyway, I'm merging this to master. Thanks again for working on this!
---
If your
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7098#issuecomment-116874844
[Test build #36043 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36043/console)
for PR 7098 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7098#issuecomment-116874930
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
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/7096#issuecomment-116830752
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7096#issuecomment-116830453
Can one of the admins verify this patch?
---
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 rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/7094#discussion_r33508154
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/ScalaReflection.scala
---
@@ -28,7 +28,8 @@ import org.apache.spark.sql.types._
*/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6921#issuecomment-116830518
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6921#issuecomment-116830478
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
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/7008#discussion_r33509760
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/Checkpoint.scala ---
@@ -44,11 +44,19 @@ class Checkpoint(@transient ssc: StreamingContext,
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7038#issuecomment-116833438
[Test build #36030 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36030/consoleFull)
for PR 7038 at commit
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/7008#discussion_r33509669
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/Checkpoint.scala ---
@@ -44,11 +44,19 @@ class Checkpoint(@transient ssc: StreamingContext,
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/7017#issuecomment-116840393
Merging into master 1.4...
---
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 JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/6959#discussion_r33510593
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeFixedWidthAggregationMap.java
---
@@ -70,67 +92,38 @@
private
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/6877#issuecomment-116845403
maybe pretty - truncate?
---
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 andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/6778#issuecomment-116845367
@yufan-liu mind closing this issue?
---
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 rxin commented on the pull request:
https://github.com/apache/spark/pull/7096#issuecomment-116845502
Thanks - LGTM. We can merge this once Jenkins comes back happy.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/7090
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7099#issuecomment-116848948
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7099#issuecomment-116849069
[Test build #36041 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36041/consoleFull)
for PR 7099 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6750#issuecomment-116849081
[Test build #36042 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36042/consoleFull)
for PR 6750 at commit
Github user aarondav commented on a diff in the pull request:
https://github.com/apache/spark/pull/5392#discussion_r33515404
--- Diff:
core/src/main/scala/org/apache/spark/deploy/client/AppClient.scala ---
@@ -40,98 +37,139 @@ import org.apache.spark.util.{ActorLogReceive,
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6920#issuecomment-116850606
[Test build #36017 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36017/console)
for PR 6920 at commit
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/7072#issuecomment-116853701
I would still argue for this change because in remote environments having a
page with a 10MB payload is pretty bad. It's just good form to compress output
for very
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7098#issuecomment-116856965
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7098#issuecomment-116856937
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
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/6959#issuecomment-116859002
LGTM overall.
---
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 aarondav commented on a diff in the pull request:
https://github.com/apache/spark/pull/5392#discussion_r33518468
--- Diff: core/src/main/scala/org/apache/spark/deploy/worker/Worker.scala
---
@@ -235,21 +250,47 @@ private[worker] class Worker(
* still not
Github user aarondav commented on a diff in the pull request:
https://github.com/apache/spark/pull/5392#discussion_r33519252
--- Diff: core/src/main/scala/org/apache/spark/deploy/worker/Worker.scala
---
@@ -302,27 +367,27 @@ private[worker] class Worker(
// the
Github user aarondav commented on a diff in the pull request:
https://github.com/apache/spark/pull/5392#discussion_r33519250
--- Diff: core/src/main/scala/org/apache/spark/deploy/worker/Worker.scala
---
@@ -302,27 +367,27 @@ private[worker] class Worker(
// the
Github user piaozhexiu commented on the pull request:
https://github.com/apache/spark/pull/6921#issuecomment-116860213
This is weird. A couple of different tests randomly fail at each run. I am
wondering whether this is because derby db intermittently fails during unit
tests.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7092#issuecomment-116860415
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
Github user aarondav commented on a diff in the pull request:
https://github.com/apache/spark/pull/5392#discussion_r33519340
--- Diff: core/src/main/scala/org/apache/spark/deploy/worker/Worker.scala
---
@@ -466,23 +531,13 @@ private[worker] class Worker(
case _ =
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6648#issuecomment-116873768
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
GitHub user davies opened a pull request:
https://github.com/apache/spark/pull/7101
[SPARK-8713] Make codegen thread safe
Codegen takes three steps:
1. Take a list of expressions, convert them into Java source code and a
list of expressions that don't not support codegen
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/6810#discussion_r33523568
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/AggregateSuite.scala ---
@@ -0,0 +1,59 @@
+/*
+ * Licensed to the Apache Software
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7070#issuecomment-116875210
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7070#issuecomment-116875229
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
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/7051#issuecomment-116875310
@JoshRosen and I discussed this offline. So apparently this feature has
been broken for a long time. Since recently, each Spark executor actually gets
its own unique
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7100#issuecomment-116875414
[Test build #36045 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36045/console)
for PR 7100 at commit
Github user marmbrus closed the pull request at:
https://github.com/apache/spark/pull/7102
---
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
401 - 500 of 1332 matches
Mail list logo