[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[srowen]

Github user srowen commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154707614
  
Yeah I meant these things all already generate a warning. I tend to agree 
with your additional change for this JIRA. Any other opinions?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[jerryshao]

Github user jerryshao commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154689758
  
Looking at the code again, looks like there's already some codes take care 
of this backward compatibility in yarn 
[client](https://github.com/apache/spark/blob/master/yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala#L1049).
 The problem is that it is not worked because `ClientArguments` is already 
initialized before this setting.

So simply we could fix this by moving this part of code into 
[here](https://github.com/apache/spark/blob/master/yarn/src/main/scala/org/apache/spark/deploy/yarn/ClientArguments.scala#L183).

```scala
// to maintain backwards-compatibility
if (!Utils.isDynamicAllocationEnabled(sparkConf)) {
  sparkConf.set("spark.executor.instances", args.numExecutors.toString)
}
```

That is more consistent with SparkSubmit, also cmd line args can override 
configuration `spark.executor.instances` if both set.

What do you think?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[jerryshao]

Github user jerryshao commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154665578
  
Ahhh, very sorry I misunderstood this PR. Yes, it is my bad to ignore this 
command line argument (`--num-executors`) which also can set executor number. 
Looks like in `SparkSubmit`, it is already changed to 
`spark.executor.instances`, so my bad there's another way which can set through 
cmd line (which is inconsistent with `SparkSubmit`).

BTW @srowen , such way of invoking yarn client already has a warning.




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[vanzin]

Github user vanzin commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154575019
  
LGTM. One day I'd like to see this duplication of command-line arguments 
vs. SparkConf entries go away...

Merging to the 3 branches.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[jerryshao]

Github user jerryshao commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154598834
  
@tgravescs , sorry for missing this parameter `--num-workers`. I suppose 
user will not directly invoke yarn client through `spark-class`. Also from 
`SparkSubmitArguments`, this `--num-workers` is not parsed. 

So maybe we should deprecate this parameter, as @vanzin said current 
there're so many ways we could set yarn related configurations, it is hard to 
manage and easy to introduce bug, also confused a lot.




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/spark/pull/9523


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[jerryshao]

Github user jerryshao commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154600765
  
Also this command is quite like a legacy code:

```
bin/spark-class org.apache.spark.deploy.yarn.Client --jar 
lib/spark-examples-1.5.2.0-hadoop2.6.0.16.1506060127.jar --class 
org.apache.spark.examples.SparkPi --num-workers 4 --worker-memory 2g 
--master-memory 1g --worker-cores 1 --queue default
```

Is it better to move forward? Compatible with old code really makes this 
part of configuration messy and inconsistent with the arguments of 
`SparkSubmit`/



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[srowen]

Github user srowen commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154651884
  
@jerryshao `--num-workers` is already deprecated and should generate a 
warning, but it happens with `--num-executors`. You mean deprecate direct use 
of `spark-class`? that should also generate a warning.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[tgravescs]

Github user tgravescs commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154471886
  
Note this should go into branch-1.6 and branch-1.5 also


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[tgravescs]

GitHub user tgravescs opened a pull request:

https://github.com/apache/spark/pull/9523

[SPARK-11555] spark on yarn spark-class --num-workers doesn't work

I tested the various options with both spark-submit and spark-class of 
specifying number of executors in both client and cluster mode where it applied.

--num-workers, --num-executors, spark.executor.instances, 
SPARK_EXECUTOR_INSTANCES, default nothing supplied

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/tgravescs/spark SPARK-11555

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/spark/pull/9523.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #9523


commit 5c104d115616f824c5048de434ce537ba1063f9f
Author: Thomas Graves 
Date:   2015-11-06T16:36:43Z

[SPARK-11555] spark on yarn spark-class --num-workers doesn't work




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154472876
  
Build started sha1 is merged.



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154472848
  
Build triggered. sha1 is merged.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[SparkQA]

Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154475270
  
**[Test build #45226 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45226/consoleFull)**
 for PR 9523 at commit 
[`5c104d1`](https://github.com/apache/spark/commit/5c104d115616f824c5048de434ce537ba1063f9f).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154475811
  

Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45226/



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[SparkQA]

Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154475802
  
**[Test build #45226 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45226/consoleFull)**
 for PR 9523 at commit 
[`5c104d1`](https://github.com/apache/spark/commit/5c104d115616f824c5048de434ce537ba1063f9f).
 * This patch **fails Scala style tests**.
 * This patch merges cleanly.
 * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154475810
  
Build finished. No test results found.



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[srowen]

Github user srowen commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154519820
  
'default' with respect to the method... dunno, it seems strange that the 
purpose of the method is to compute a number of executors to use but it takes 
this as a param. Really it's the value to fall back on unless otherwise 
specified.

I don't feel strongly; the fix is right.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[SparkQA]

Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154520921
  
**[Test build #1997 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1997/consoleFull)**
 for PR 9523 at commit 
[`e32ad30`](https://github.com/apache/spark/commit/e32ad30369c87c32cc99d26548db7d9a4956b4de).
 * This patch **fails Scala style tests**.
 * This patch merges cleanly.
 * This patch adds the following public classes _(experimental)_:\n  * 
`final class ShuffleSortDataFormat extends SortDataFormat `\n  * `final class UnsafeSortDataFormat extends 
SortDataFormat `\n


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154520869
  
Build finished. No test results found.



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[SparkQA]

Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154520866
  
**[Test build #45247 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45247/consoleFull)**
 for PR 9523 at commit 
[`e32ad30`](https://github.com/apache/spark/commit/e32ad30369c87c32cc99d26548db7d9a4956b4de).
 * This patch **fails Scala style tests**.
 * This patch merges cleanly.
 * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[srowen]

Github user srowen commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154516259
  
This looks like the right fix (modulo the long line). Should the param be 
something like a "current num executors" or a "default"? its default value is a 
default value after all.

It sort of seems like `--num-executors` could be used to set the initial 
num executors under dynamic allocation if not already set, but maybe that's 
best left untouched.

CC @jerryshao 


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154519921
  
Build triggered. sha1 is merged.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154519949
  
Build started sha1 is merged.



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154520871
  

Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45247/



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[tgravescs]

Github user tgravescs commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154519370
  
I fixed the scala style error.  I'm fine with changing the name of the 
parameter but don't like default as its not always the default.  It could be 
what the user specified.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[SparkQA]

Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154520206
  
**[Test build #45247 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45247/consoleFull)**
 for PR 9523 at commit 
[`e32ad30`](https://github.com/apache/spark/commit/e32ad30369c87c32cc99d26548db7d9a4956b4de).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[SparkQA]

Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154520306
  
**[Test build #1997 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1997/consoleFull)**
 for PR 9523 at commit 
[`e32ad30`](https://github.com/apache/spark/commit/e32ad30369c87c32cc99d26548db7d9a4956b4de).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154544393
  
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 this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154544363
  
 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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[SparkQA]

Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154545932
  
**[Test build #45254 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45254/consoleFull)**
 for PR 9523 at commit 
[`22b1bad`](https://github.com/apache/spark/commit/22b1badd54cb2e0ff7e71e6b1a75a40adfe069dd).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[SparkQA]

Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154553412
  
**[Test build #45254 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45254/consoleFull)**
 for PR 9523 at commit 
[`22b1bad`](https://github.com/apache/spark/commit/22b1badd54cb2e0ff7e71e6b1a75a40adfe069dd).
 * This patch passes all tests.
 * This patch merges cleanly.
 * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-11555] spark on yarn spark-class --num-...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9523#issuecomment-154553615
  
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org