[GitHub] spark pull request: [MINOR][DOCS] Fixes markup so code is properly...

2015-09-17 Thread srowen
Github user srowen commented on the pull request:

https://github.com/apache/spark/pull/8795#issuecomment-141053254
  
Perfect, yes, just modify the title of this PR to connect them. See 
https://cwiki.apache.org/confluence/display/SPARK/Contributing+to+Spark 


---
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][DOCS] Fixes markup so code is properly...

2015-09-17 Thread jaceklaskowski
Github user jaceklaskowski commented on the pull request:

https://github.com/apache/spark/pull/8795#issuecomment-141048359
  
Is [SPARK-10662](https://issues.apache.org/jira/browse/SPARK-10662) what 
you were thinking about? What should be the next steps? Guide me in the jira, 
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 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][DOCS] Fixes markup so code is properly...

2015-09-17 Thread srowen
Github user srowen commented on the pull request:

https://github.com/apache/spark/pull/8795#issuecomment-141019047
  
This is a good fix, but this is another instance where I suspect the same 
issue exists in several markup files, like `configuration.html`. It's worth a 
JIRA since I think catching and fixing all of these is one non-trivial logical 
change.

If you can, avoid whitespace changes like stripping or adding space at the 
end of lines. It just adds to the diff and makes for a tiny extra chance of 
merge conflicts.


---
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][DOCS] Fixes markup so code is properly...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/8795#issuecomment-140998628
  
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 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][DOCS] Fixes markup so code is properly...

2015-09-17 Thread jaceklaskowski
GitHub user jaceklaskowski opened a pull request:

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

[MINOR][DOCS] Fixes markup so code is properly formatted

* Backticks are processed properly in Spark Properties table
* Removed unnecessary spaces
* See 
http://people.apache.org/~pwendell/spark-nightly/spark-master-docs/latest/running-on-yarn.html

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

$ git pull https://github.com/jaceklaskowski/spark docs-yarn-formatting

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

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


commit 90c9a96720f97be012e975406c2f861af3c7c6a0
Author: Jacek Laskowski 
Date:   2015-09-17T07:54:20Z

[MINOR][DOCS] Fixes markup so code is properly formatted

* Backticks are processed properly in Spark Properties table
* Removed unnecessary spaces
* See 
http://people.apache.org/~pwendell/spark-nightly/spark-master-docs/latest/running-on-yarn.html




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