Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15607
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/15607
**[Test build #67441 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67441/consoleFull)**
for PR 15607 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15607
**[Test build #67441 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67441/consoleFull)**
for PR 15607 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/15607
Hmm, it works fine with
`roxygen2 "roxygen2" "/usr/local/lib/R/site-library" "5.0.1"`
maybe it is the older version on Jenkins?
```
First time using roxygen2 4.0.
Github user tejasapatil closed the pull request at:
https://github.com/apache/spark/pull/15605
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15428
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67429/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15428
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67430/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15428
**[Test build #67429 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67429/consoleFull)**
for PR 15428 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15428
**[Test build #67430 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67430/consoleFull)**
for PR 15428 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15608
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67440/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15608
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/15608
**[Test build #67440 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67440/consoleFull)**
for PR 15608 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15597
**[Test build #67433 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67433/consoleFull)**
for PR 15597 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15597
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67433/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15597
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 HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15609
Hm, does this cause a actual problem? I thought this is covered in
type-widening.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13891
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/13891
**[Test build #67435 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67435/consoleFull)**
for PR 13891 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13891
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67435/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15606
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/15606
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67437/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15606
**[Test build #67437 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67437/consoleFull)**
for PR 15606 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15608
**[Test build #67440 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67440/consoleFull)**
for PR 15608 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15607
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/15607
**[Test build #67442 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67442/consoleFull)**
for PR 15607 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15607
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67442/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15607
**[Test build #67442 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67442/consoleFull)**
for PR 15607 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15541
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67428/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15541
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 mortada commented on the issue:
https://github.com/apache/spark/pull/15053
@HyukjinKwon I've created a JIRA ticket and also went through all the files
you mentioned, please take a look
---
If your project is set up for it, you can reply to this email and have your
reply
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15607
**[Test build #67443 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67443/consoleFull)**
for PR 15607 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15607
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/15607
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67443/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15566
**[Test build #67439 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67439/consoleFull)**
for PR 15566 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15608
cc @felixcheung I recall we talked about this before. I first wanted to
handle all the argument type checking but I just decided to do what I am pretty
sure of (I remember we were concern of
GitHub user HyukjinKwon opened a pull request:
https://github.com/apache/spark/pull/15608
[SPARK-17838][SparkR] Check named arguments for options and use formatted R
friendly message from JVM exception message
## What changes were proposed in this pull request?
This PR
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15607
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67441/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15541
**[Test build #67428 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67428/consoleFull)**
for PR 15541 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15607
**[Test build #67443 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67443/consoleFull)**
for PR 15607 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15609
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 priyankagargnitk opened a pull request:
https://github.com/apache/spark/pull/15609
[SPARK-18048][SQL] To make behaviour of If consistent, in case of true
expression and false expression are of compatible data types.
## What changes were proposed in this pull request?
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15428
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/15428
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 HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15053
@srowen Do you mind if I ask to trigger the build via Jenkins if it looks
reasonable (or maybe "ok to test")? Let me try to take a look closer but I
would like @holdenk also to take a look
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15606
This bug only affects the complex type, (`ArrayType`, `StructType`, and
`MapType`)?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user priyankagargnitk commented on the issue:
https://github.com/apache/spark/pull/15609
actually, its not covered in type widening... So, if any of my expression
is calling if expression ( in nested) .. its failing because of this issue.
---
If your project is set up for it,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15553
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/15553
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67436/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15566
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67439/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15566
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 srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15603#discussion_r84640346
--- Diff: core/src/main/scala/org/apache/spark/ui/SparkUI.scala ---
@@ -93,15 +91,15 @@ private[spark] class SparkUI private (
/** Stop the server
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15603#discussion_r84640931
--- Diff: core/src/main/scala/org/apache/spark/ui/SparkUI.scala ---
@@ -135,7 +133,7 @@ private[spark] class SparkUI private (
private[spark] abstract
Github user NarineK commented on the issue:
https://github.com/apache/spark/pull/10162
I am trying different ways to solve the problem without renaming the
columns and it seems that a better place to change the column names would be
here:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15377
**[Test build #67445 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67445/consoleFull)**
for PR 15377 at commit
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15377#discussion_r84643899
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -2508,6 +2508,27 @@ private[spark] object Utils extends Logging {
}
}
Github user priyankagargnitk commented on the issue:
https://github.com/apache/spark/pull/15609
Actually our use case is little different, we are not invoking it with the
Select queries.. We are using JS to let user type expressions and then we have
created out own layer on top of
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/15598#discussion_r84643826
--- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala ---
@@ -1059,9 +1059,11 @@ private[spark] class Client(
} catch {
Github user jaceklaskowski commented on the issue:
https://github.com/apache/spark/pull/15603
The build has just proved that my thinking was correct. I think I'll
propose few other (more agressive) changes to clean the code up a little bit
more. I'd appreciate any comments (or
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15553
**[Test build #67436 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67436/consoleFull)**
for PR 15553 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15568
**[Test build #67444 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67444/consoleFull)**
for PR 15568 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15566
**[Test build #67439 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67439/consoleFull)**
for PR 15566 at commit
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/15588
I think this fix cannot really handle this imbalance receiver allocation
problem, also blindly waste the CPU time.
What @lw-lin mentioned is a feasible solution to wait for executors to
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15513
@HyukjinKwon @gatorsmile on the question just above, I'm neutral. I don't
think either style is particularly better. All else equal, leave it as is, as
we've got a ton of changes in flight here.
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15609
@priyankagargnitk Could I please ask some codes which I can reproduce this
issue? It seems fine the sql below:
```sql
spark-sql> SELECT if(true, cast('2014-01-01' as date),
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15598#discussion_r84644113
--- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala ---
@@ -1059,9 +1059,11 @@ private[spark] class Client(
} catch {
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15609
Then, wouldn't the responsibility of type coercion be your own layer? I
guess this PR virtually mean every expression handled in `TypeCoercion` should
be fixed too. For example, I guess
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15515
**[Test build #67468 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67468/consoleFull)**
for PR 15515 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15515
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67468/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15515
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/15441
**[Test build #67471 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67471/consoleFull)**
for PR 15441 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15463
---
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 tdas commented on the issue:
https://github.com/apache/spark/pull/12358
@agsachin Any thoughts on @ScrapCodes comment? There are unrelated code
changes and best to remove.
And if you are unable to make these change, mind closing the PR?
---
If your project is
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/15535
@tdas could you close this PR? 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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/15607
**[Test build #67476 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67476/consoleFull)**
for PR 15607 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15607
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 jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/12195#discussion_r84809994
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/StreamingLogisticRegression.scala
---
@@ -1,75 +0,0 @@
-/*
- * Licensed to the
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/12195#discussion_r84810043
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/TallSkinnySVD.scala ---
@@ -1,62 +0,0 @@
-/*
- * Licensed to the Apache Software
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/12195#discussion_r84809881
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/StreamingLinearRegression.scala
---
@@ -1,73 +0,0 @@
-/*
- * Licensed to the
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/12195#discussion_r84810032
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/TallSkinnyPCA.scala ---
@@ -1,62 +0,0 @@
-/*
- * Licensed to the Apache Software
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15596
**[Test build #67480 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67480/consoleFull)**
for PR 15596 at commit
Github user sarutak commented on the issue:
https://github.com/apache/spark/pull/12257
@ajbozarth @tdas Thanks for your interest. I'll rebase this to master and
reflect the comments.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/15566#discussion_r84816423
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/InsertIntoHiveTableSuite.scala
---
@@ -370,17 +370,6 @@ class InsertIntoHiveTableSuite extends
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/13259
@zsxwing Could you take a look at this PR. Would be good to merge this.
---
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/15009#discussion_r84789919
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/SparkLauncher.java ---
@@ -528,13 +581,41 @@ public SparkAppHandle
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15009#discussion_r84789427
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/SparkLauncher.java ---
@@ -73,6 +74,14 @@
/** Logger name to use when launching a child
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15009#discussion_r84791881
--- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala ---
@@ -1186,9 +1209,13 @@ private[spark] class Client(
}
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15009#discussion_r84788565
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/LauncherProtocol.java ---
@@ -32,6 +32,9 @@
/** Environment variable where the secret for
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15009#discussion_r84782803
--- Diff: core/src/main/scala/org/apache/spark/SparkApp.scala ---
@@ -0,0 +1,29 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15009#discussion_r84789538
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/SparkLauncher.java ---
@@ -107,6 +123,27 @@ public static void setConfig(String name, String
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15009#discussion_r84785459
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -710,7 +723,11 @@ object SparkSubmit {
printWarning("Subclasses of
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15009#discussion_r84789777
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/SparkLauncher.java ---
@@ -107,6 +123,27 @@ public static void setConfig(String name, String
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15009#discussion_r84787477
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/AbstractSparkAppHandle.java ---
@@ -0,0 +1,133 @@
+/*
+ * Licensed to the Apache Software
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15009#discussion_r84788125
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/ChildProcAppHandle.java ---
@@ -103,67 +46,22 @@ public synchronized void kill() {
try
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15009#discussion_r84791767
--- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala ---
@@ -645,11 +664,13 @@ private[spark] class Client(
remoteFs,
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15009#discussion_r84790527
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/SparkSubmitRunner.java ---
@@ -0,0 +1,59 @@
+/*
+ * Licensed to the Apache Software
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15009#discussion_r84786490
--- Diff:
core/src/main/scala/org/apache/spark/launcher/LauncherBackend.scala ---
@@ -17,38 +17,71 @@
package org.apache.spark.launcher
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15009#discussion_r84785044
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -710,7 +723,11 @@ object SparkSubmit {
printWarning("Subclasses of
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15009#discussion_r84791332
--- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala ---
@@ -56,23 +56,38 @@ import org.apache.spark.internal.config._
import
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15009#discussion_r84788429
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/ChildThreadAppHandle.java ---
@@ -0,0 +1,53 @@
+/*
+ * Licensed to the Apache Software
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15009#discussion_r84782648
--- Diff: core/src/main/scala/org/apache/spark/SparkApp.scala ---
@@ -0,0 +1,29 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
1 - 100 of 602 matches
Mail list logo
