GitHub user jerryshao opened a pull request:
https://github.com/apache/spark/pull/19074
[SPARK-21714][CORE][BACKPORT-2.2] Avoiding re-uploading remote resources in
yarn client mode
## What changes were proposed in this pull request?
This is a backport PR to fix issue of
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17357
**[Test build #81200 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81200/testReport)**
for PR 17357 at commit
Github user ScrapCodes commented on a diff in the pull request:
https://github.com/apache/spark/pull/17357#discussion_r135699350
--- Diff:
core/src/main/scala/org/apache/spark/deploy/rest/StandaloneRestServer.scala ---
@@ -139,7 +139,9 @@ private[rest] class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19048
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19048
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81195/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19048
**[Test build #81195 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81195/testReport)**
for PR 19048 at commit
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/18317#discussion_r135694280
--- Diff: core/src/main/java/org/apache/spark/io/ReadAheadInputStream.java
---
@@ -0,0 +1,317 @@
+/*
+ * Licensed under the Apache License, Version
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/18317#discussion_r135694119
--- Diff: core/src/main/java/org/apache/spark/io/ReadAheadInputStream.java
---
@@ -0,0 +1,317 @@
+/*
+ * Licensed under the Apache License, Version
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/18317#discussion_r135695472
--- Diff: core/src/main/java/org/apache/spark/io/ReadAheadInputStream.java
---
@@ -0,0 +1,292 @@
+/*
+ * Licensed under the Apache License, Version
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/18317#discussion_r135695625
--- Diff: core/src/main/java/org/apache/spark/io/ReadAheadInputStream.java
---
@@ -0,0 +1,292 @@
+/*
+ * Licensed under the Apache License, Version
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/18317#discussion_r135693340
--- Diff: core/src/main/java/org/apache/spark/io/ReadAheadInputStream.java
---
@@ -0,0 +1,317 @@
+/*
+ * Licensed under the Apache License, Version
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17014#discussion_r135695930
--- Diff: mllib/src/main/scala/org/apache/spark/ml/Predictor.scala ---
@@ -85,6 +86,10 @@ abstract class Predictor[
M <:
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/18317#discussion_r135692852
--- Diff: core/src/main/java/org/apache/spark/io/ReadAheadInputStream.java
---
@@ -0,0 +1,292 @@
+/*
+ * Licensed under the Apache License, Version
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19055
**[Test build #81199 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81199/testReport)**
for PR 19055 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19072
**[Test build #81198 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81198/testReport)**
for PR 19072 at commit
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/18902
@yanboliang Although dispointed by DF's performance, I also approve the
choice of DF just for less code.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19055#discussion_r135692633
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -322,6 +322,15 @@ object SQLConf {
.booleanConf
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/18317#discussion_r135692546
--- Diff: core/src/main/java/org/apache/spark/io/ReadAheadInputStream.java
---
@@ -0,0 +1,292 @@
+/*
+ * Licensed under the Apache License, Version
Github user zhengruifeng commented on a diff in the pull request:
https://github.com/apache/spark/pull/17014#discussion_r135692470
--- Diff: mllib/src/main/scala/org/apache/spark/ml/Predictor.scala ---
@@ -85,6 +86,10 @@ abstract class Predictor[
M <:
Github user sitalkedia commented on a diff in the pull request:
https://github.com/apache/spark/pull/18317#discussion_r135689398
--- Diff: core/src/main/java/org/apache/spark/io/ReadAheadInputStream.java
---
@@ -0,0 +1,292 @@
+/*
+ * Licensed under the Apache License,
Github user sitalkedia commented on a diff in the pull request:
https://github.com/apache/spark/pull/18317#discussion_r135689367
--- Diff: core/src/main/java/org/apache/spark/io/ReadAheadInputStream.java
---
@@ -0,0 +1,292 @@
+/*
+ * Licensed under the Apache License,
Github user sitalkedia commented on a diff in the pull request:
https://github.com/apache/spark/pull/18317#discussion_r135689239
--- Diff: core/src/main/java/org/apache/spark/io/ReadAheadInputStream.java
---
@@ -0,0 +1,292 @@
+/*
+ * Licensed under the Apache License,
Github user sitalkedia commented on a diff in the pull request:
https://github.com/apache/spark/pull/18317#discussion_r135689225
--- Diff: core/src/main/java/org/apache/spark/io/ReadAheadInputStream.java
---
@@ -0,0 +1,292 @@
+/*
+ * Licensed under the Apache License,
Github user sitalkedia commented on a diff in the pull request:
https://github.com/apache/spark/pull/18317#discussion_r135689194
--- Diff: core/src/main/java/org/apache/spark/io/ReadAheadInputStream.java
---
@@ -0,0 +1,292 @@
+/*
+ * Licensed under the Apache License,
Github user sitalkedia commented on a diff in the pull request:
https://github.com/apache/spark/pull/18317#discussion_r135689169
--- Diff: core/src/main/java/org/apache/spark/io/ReadAheadInputStream.java
---
@@ -0,0 +1,292 @@
+/*
+ * Licensed under the Apache License,
Github user sitalkedia commented on a diff in the pull request:
https://github.com/apache/spark/pull/18317#discussion_r135689113
--- Diff: core/src/main/java/org/apache/spark/io/ReadAheadInputStream.java
---
@@ -0,0 +1,292 @@
+/*
+ * Licensed under the Apache License,
Github user sitalkedia commented on a diff in the pull request:
https://github.com/apache/spark/pull/18317#discussion_r135689067
--- Diff: core/src/main/java/org/apache/spark/io/ReadAheadInputStream.java
---
@@ -0,0 +1,292 @@
+/*
+ * Licensed under the Apache License,
Github user sitalkedia commented on a diff in the pull request:
https://github.com/apache/spark/pull/18317#discussion_r135688838
--- Diff: core/src/main/java/org/apache/spark/io/ReadAheadInputStream.java
---
@@ -0,0 +1,292 @@
+/*
+ * Licensed under the Apache License,
Github user sitalkedia commented on a diff in the pull request:
https://github.com/apache/spark/pull/18317#discussion_r135688946
--- Diff: core/src/main/java/org/apache/spark/io/ReadAheadInputStream.java
---
@@ -0,0 +1,292 @@
+/*
+ * Licensed under the Apache License,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19073
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 project does not have this
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18317
**[Test build #81197 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81197/testReport)**
for PR 18317 at commit
GitHub user caneGuy opened a pull request:
https://github.com/apache/spark/pull/19073
[SPARK][DEPLOY] Should print error when upload same file multiple timâ¦
â¦es to yarn
## What changes were proposed in this pull request?
Now when submit job with yarn,and upload
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19072
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81196/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19072
**[Test build #81196 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81196/testReport)**
for PR 19072 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19072
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19064
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81192/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19064
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/19064
**[Test build #81192 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81192/testReport)**
for PR 19064 at commit
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19055#discussion_r135686725
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -322,6 +322,15 @@ object SQLConf {
.booleanConf
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19072
**[Test build #81196 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81196/testReport)**
for PR 19072 at commit
GitHub user WeichenXu123 opened a pull request:
https://github.com/apache/spark/pull/19072
[SPARK-17133][ML][FOLLOW-UP] Add convenient method `asBinary` for casting
to BinaryLogisticRegressionSummary
## What changes were proposed in this pull request?
add an "asBinary"
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/17862
Catching up here... To make sure I caught the decisions made in the
discussion above, is it correct that this PR will:
* Add support for squared hinge loss, and use that as the default (which
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19048
**[Test build #81195 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81195/testReport)**
for PR 19048 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19048
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19048
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81194/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19048
**[Test build #81194 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81194/testReport)**
for PR 19048 at commit
Github user guoxiaolongzte closed the pull request at:
https://github.com/apache/spark/pull/19049
---
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
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/19049
Ok, Thank you for your comments.
---
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 guoxiaolongzte closed the pull request at:
https://github.com/apache/spark/pull/18917
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19048
**[Test build #81194 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81194/testReport)**
for PR 19048 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19048
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19048
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81193/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19048
**[Test build #81193 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81193/testReport)**
for PR 19048 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19048
**[Test build #81193 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81193/testReport)**
for PR 19048 at commit
Github user sitalkedia commented on the issue:
https://github.com/apache/spark/pull/19048
@jiangxb1987 - I agree with you. I do not have the context or history to
comment on that. Unfortunately, the api has been designed that way and book
keeping of target number of executors is done
Github user heary-cao commented on the issue:
https://github.com/apache/spark/pull/19069
@sameeragarwalï¼Thank you for your suggest, modify it.
---
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 WeichenXu123 closed the pull request at:
https://github.com/apache/spark/pull/19026
---
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
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/19018
@felixcheung In Jenkins Log I only found Random forest and Decision Tree
failed, random forest failed more frequently. thanks!
---
If your project is set up for it, you can reply to this
Github user yaooqinn commented on a diff in the pull request:
https://github.com/apache/spark/pull/19068#discussion_r135676677
--- Diff:
sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/SparkSQLCLIDriver.scala
---
@@ -81,11 +81,7 @@ private[hive] object
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19061
>If such a thing as a non-Spark repl-like application exists, it wouldn't
be getting the progress bar by default, for example, because its default log
level is "INFO" in Spark, something that
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/18962
Sorry I missed the comments, I will file another PR against branch 2.2.
---
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 HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18111
OK. LGTM. I think this anyway should issue warnings:
```
WARN org.apache.hadoop.mapreduce.lib.output.FileOutputCommitter: Output
Path is null in setupJob()
WARN
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19064
**[Test build #81192 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81192/testReport)**
for PR 19064 at commit
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/17357
LGTM, cc @cloud-fan
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17357
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/17357
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81191/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17357
**[Test build #81191 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81191/testReport)**
for PR 17357 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19061
I see. Thank you for the guide!
---
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 vanzin commented on the issue:
https://github.com/apache/spark/pull/19061
> Is it okay we change this behavior like this?
It's not optimal, but at the same time, the existing behavior wasn't really
correctly advertised anyway. If such a thing as a non-Spark
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/16422
@gatorsmile Will update in this week.
---
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 wzhfy commented on the issue:
https://github.com/apache/spark/pull/18421
@mbasmanova Great work! I was really busy in the past two months so I
didn't have time to look at this.
Thank @gatorsmile for reviewing and merging this PR!
---
If your project is set up for
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19061
BTW, @vanzin . I was waiting your comment because @jerryshao is worried
about 3rd party relations.
> other repl-like projects may actually require this
Is it okay we change this
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19061
Thank you for reiview, @vanzin . Yep. @jerryshao also advised me the
missing point. I'll try to make it more general.
---
If your project is set up for it, you can reply to this email and
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19061#discussion_r135663178
--- Diff: docs/configuration.md ---
@@ -804,7 +804,7 @@ Apart from these, the following properties are also
available, and may be useful
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19061#discussion_r135663076
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -537,6 +537,11 @@ object SparkSubmit extends CommandLineUtils {
}
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/19050
also cc @cloud-fan for review. 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 have this feature
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/19050
ping @hvanhovell Does the current change look good for you?
---
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 issue:
https://github.com/apache/spark/pull/17980
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81190/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17980
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/17980
**[Test build #81190 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81190/testReport)**
for PR 17980 at commit
Github user smurching commented on a diff in the pull request:
https://github.com/apache/spark/pull/19065#discussion_r135655220
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/ProbabilisticClassifierSuite.scala
---
@@ -91,4 +94,54 @@ object
Github user smurching commented on a diff in the pull request:
https://github.com/apache/spark/pull/19065#discussion_r135653663
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/ProbabilisticClassifierSuite.scala
---
@@ -91,4 +94,54 @@ object
Github user smurching commented on a diff in the pull request:
https://github.com/apache/spark/pull/19065#discussion_r135656421
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/ProbabilisticClassifierSuite.scala
---
@@ -91,4 +94,54 @@ object
Github user smurching commented on a diff in the pull request:
https://github.com/apache/spark/pull/19065#discussion_r135653421
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/ProbabilisticClassifierSuite.scala
---
@@ -18,7 +18,10 @@
package
Github user smurching commented on a diff in the pull request:
https://github.com/apache/spark/pull/19065#discussion_r135653044
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/DecisionTreeClassifierSuite.scala
---
@@ -262,6 +262,9 @@ class
Github user smurching commented on a diff in the pull request:
https://github.com/apache/spark/pull/19065#discussion_r135653479
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/ProbabilisticClassifierSuite.scala
---
@@ -18,7 +18,10 @@
package
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19049
> For the 'Spark User' column. It would be identical in almost all use
cases.
That is absolutely not true in a true multi-user environment.
---
If your project is set up for it, you can
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/19064
I'm +1 on this, it should be useful to pattern match the trait instead of
handling ScalaUDF/ScalaUDAF/PythonUDF... in future optimize rules.
---
If your project is set up for it, you can reply
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/18731
The CI error actually is a timeout. I don't know the reason of the timeout
or how to retrigger it. If I should do anything please just tell me.
If there is anything I can do for this PR just
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/19071
CC @WeichenXu123
---
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,
Github user sameeragarwal commented on a diff in the pull request:
https://github.com/apache/spark/pull/19069#discussion_r135645027
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/JoinBenchmark.scala
---
@@ -212,18 +212,17 @@ class JoinBenchmark extends
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/19064
I'm +1 on this, currently we are pattern matching `ScalaUDF` in optimize
rules which is suboptional.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19071
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81189/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19071
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/19071
**[Test build #81189 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81189/testReport)**
for PR 19071 at commit
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18837#discussion_r135640958
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosClusterScheduler.scala
---
@@ -383,15 +385,57 @@
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18837#discussion_r135641205
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/deploy/mesos/config.scala
---
@@ -58,12 +58,43 @@ package object config {
Github user JoshRosen commented on the issue:
https://github.com/apache/spark/pull/19056
(Dummy comment to test JIRA linkage)
---
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/17357
**[Test build #81191 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81191/testReport)**
for PR 17357 at commit
Github user sitalkedia commented on the issue:
https://github.com/apache/spark/pull/18317
Sure, I will address @mridulm comment in next few days.
---
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
1 - 100 of 253 matches
Mail list logo
