[GitHub] spark pull request: Minor change in the comment of spark-defaults....

2014-11-24 Thread dbtsai
Github user dbtsai closed the pull request at:

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


---
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: Minor change in the comment of spark-defaults....

2014-10-19 Thread dbtsai
Github user dbtsai commented on the pull request:

https://github.com/apache/spark/pull/2709#issuecomment-59667207
  
@andrewor14 Sorry for late reply since I was on vacation in Europe last 
week. I can continue work on this after I finish my talk in IOTA conf tomorrow. 


---
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: Minor change in the comment of spark-defaults....

2014-10-17 Thread andrewor14
Github user andrewor14 commented on the pull request:

https://github.com/apache/spark/pull/2709#issuecomment-59596156
  
Hey @dbtsai can you update this now that #2379 has gone in? In particular 
this is now used by the Spark daemons too (i.e. Worker, Master, HistoryServer). 
I'm don't feel strongly about commenting that this applies to spark-shell and 
pyspark one way or the other. If you're occupied with other things I can also 
take this over if you wish.


---
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: Minor change in the comment of spark-defaults....

2014-10-08 Thread dbtsai
GitHub user dbtsai opened a pull request:

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

Minor change in the comment of spark-defaults.conf.template

spark-defaults.conf is used in spark-shell as well, and this PR added this 
into the comment.

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

$ git pull https://github.com/dbtsai/spark docs

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

https://github.com/apache/spark/pull/2709.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 #2709


commit b3e1ff1b808380707d04277c2379bf5b03556662
Author: DB Tsai dbt...@alpinenow.com
Date:   2014-10-08T08:53:25Z

add spark-shell




---
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: Minor change in the comment of spark-defaults....

2014-10-08 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/2709#issuecomment-58329045
  
  [QA tests have 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/21457/consoleFull)
 for   PR 2709 at commit 
[`b3e1ff1`](https://github.com/apache/spark/commit/b3e1ff1b808380707d04277c2379bf5b03556662).
 * This patch merges cleanly.


---
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: Minor change in the comment of spark-defaults....

2014-10-08 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/2709#issuecomment-58336192
  
  [QA tests have 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/21457/consoleFull)
 for   PR 2709 at commit 
[`b3e1ff1`](https://github.com/apache/spark/commit/b3e1ff1b808380707d04277c2379bf5b03556662).
 * 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: Minor change in the comment of spark-defaults....

2014-10-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/2709#issuecomment-58336202
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 

https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/21457/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



[GitHub] spark pull request: Minor change in the comment of spark-defaults....

2014-10-08 Thread andrewor14
Github user andrewor14 commented on the pull request:

https://github.com/apache/spark/pull/2709#issuecomment-58419538
  
Well, `spark-shell` goes through `spark-submit`. Actually as of a recent PR 
(#2379) that will be merged soon, this file will be used for other Spark 
daemons too (Worker, Master, HistoryServer...). Once that's merged we should 
generalize this comment to reflect that.


---
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