[GitHub] spark issue #19996: [MINOR][DOC] Fix the link of 'Getting Started'

2017-12-17 Thread srowen
Github user srowen commented on the issue:

https://github.com/apache/spark/pull/19996
  
Merged to master


---

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



[GitHub] spark issue #19996: [MINOR][DOC] Fix the link of 'Getting Started'

2017-12-17 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/19996
  
**[Test build #4012 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4012/testReport)**
 for PR 19996 at commit 
[`029f676`](https://github.com/apache/spark/commit/029f676fec5e409f8711679a1ab4689980b9f22a).
 * This patch passes all tests.
 * This patch merges cleanly.
 * This patch adds no public classes.


---

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



[GitHub] spark issue #19996: [MINOR][DOC] Fix the link of 'Getting Started'

2017-12-17 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/19996
  
**[Test build #4012 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4012/testReport)**
 for PR 19996 at commit 
[`029f676`](https://github.com/apache/spark/commit/029f676fec5e409f8711679a1ab4689980b9f22a).


---

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



[GitHub] spark issue #19996: [MINOR][DOC] Fix the link of 'Getting Started'

2017-12-17 Thread mcavdar
Github user mcavdar commented on the issue:

https://github.com/apache/spark/pull/19996
  
@srowen 
[Here](https://github.com/mcavdar/NLP/blob/master/Broken%20Links/spark/spark_404links.txt)
 is all broken links, it may be useful. Each line contains broken link and 
parent page (separated by tab). About 75-100 broken links are related to 
"http(s)://spark.apache.org/docs/latest".


---

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



[GitHub] spark issue #19996: [MINOR][DOC] Fix the link of 'Getting Started'

2017-12-15 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue:

https://github.com/apache/spark/pull/19996
  
Oh, maybe it's not quite related but do you mind if I ask to fix the below 
too?


https://github.com/apache/spark/blob/ccdf21f56e4ff5497d7770dcbee2f7a60bb9e3a7/docs/sql-programming-guide.md#L501-L504

to (just adding a newline)

```
  
  

 ### Run SQL on files directly 
```

because it currently breaks doc rendering as below:

https://user-images.githubusercontent.com/6477701/31481516-cd9ddb80-af5e-11e7-970b-d2c279f025d4.png;
 width="200" />



---

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



[GitHub] spark issue #19996: [MINOR][DOC] Fix the link of 'Getting Started'

2017-12-15 Thread mcavdar
Github user mcavdar commented on the issue:

https://github.com/apache/spark/pull/19996
  
It has already been fixed. 
https://github.com/apache/spark/commit/01f183e8497d4931f1fe5c69ff16fe84b1e41492
I'm running a link checker to find and fix more.


---

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



[GitHub] spark issue #19996: [MINOR][DOC] Fix the link of 'Getting Started'

2017-12-15 Thread srowen
Github user srowen commented on the issue:

https://github.com/apache/spark/pull/19996
  
This is fine @mcavdar . While we're at it, there's a link to 
https://mesos.apache.org/documentation/latest/mesos-frameworks/ that looks dead 
too -- do you know where that goes?

Also -- just a pet peeve of mine -- I think we can use HTTPS URLs wherever 
supported. Not worth changing for its own sake, but might be worth it if 
changing these links for other reasons.

And you're welcome to run a link checker to discover more dead links. I 
know there are more.


---

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



[GitHub] spark issue #19996: [MINOR][DOC] Fix the link of 'Getting Started'

2017-12-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/19996
  
Can one of the admins verify this patch?


---

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



[GitHub] spark issue #19996: [MINOR][DOC] Fix the link of 'Getting Started'

2017-12-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/19996
  
Can one of the admins verify this patch?


---

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