[GitHub] spark pull request: [MINOR][DOCS] quick-start: Escape shell comman...

2015-11-14 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/9647#issuecomment-156709368 I can't close your PR, so please close it yourself. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: [MINOR][DOCS] quick-start: Escape shell comman...

2015-11-14 Thread Tarrasch
Github user Tarrasch commented on the pull request: https://github.com/apache/spark/pull/9647#issuecomment-156706664 I'm not going to go through all docs and update them. That this simple patch have spawned so much discussion is not worth my time. Clearly this increases

[GitHub] spark pull request: [MINOR][DOCS] quick-start: Escape shell comman...

2015-11-14 Thread Tarrasch
Github user Tarrasch closed the pull request at: https://github.com/apache/spark/pull/9647 --- 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] spark pull request: [MINOR][DOCS] quick-start: Escape shell comman...

2015-11-14 Thread Tarrasch
Github user Tarrasch commented on the pull request: https://github.com/apache/spark/pull/9647#issuecomment-156769218 @srowen sent me an email pointing out I could have a better manner. He is of course correct and I thank him for his time. :) As for this PR. Let's close it as

[GitHub] spark pull request: [MINOR][DOCS] quick-start: Escape shell comman...

2015-11-13 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/9647#issuecomment-156362518 Here, we are talking about modifying shell commands presented in docs, not in `.sh` files. Your point is that the shell _does_ matter right? The `.sh` files in

[GitHub] spark pull request: [MINOR][DOCS] quick-start: Escape shell comman...

2015-11-12 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/9647#issuecomment-156084719 I don't know that in general Spark scripts work at all with `zsh`. Why single quote and not double? I think there are many more instances you'd have to change too, not

[GitHub] spark pull request: [MINOR][DOCS] quick-start: Escape shell comman...

2015-11-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9647#issuecomment-155984484 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

[GitHub] spark pull request: [MINOR][DOCS] quick-start: Escape shell comman...

2015-11-11 Thread Tarrasch
GitHub user Tarrasch opened a pull request: https://github.com/apache/spark/pull/9647 [MINOR][DOCS] quick-start: Escape shell commands When using the (very popular) zsh shell, running commands like `echo local[4]` will result in an error as the brackets will be interpreted by the