GitHub user jarrettmeyer opened a pull request:
https://github.com/apache/spark/pull/17861
Remove excess quotes in Windows executable
## What changes were proposed in this pull request?
Quotes are already added to the RUNNER variable on line 54. There is no
need to put quotes on line 67. If you do, you will get an error when launching
Spark.
'""C:\Program' is not recognized as an internal or external command,
operable program or batch file.
## How was this patch tested?
Tested manually on Windows 10.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarrettmeyer/spark fix-windows-cmd
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/17861.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 #17861
----
commit d784a3101338634d022b32dcd950f78351b69d29
Author: Jarrett Meyer <[email protected]>
Date: 2017-05-04T18:52:09Z
Remove excess quotes in Windows executable
Quotes are already added to the RUNNER variable on line 54. There is no
need to put quotes on line 67.
If you have 2 sets of quotes, you receive the following error:
'""C:\Program' is not recognized as an internal or external command,
operable program or batch file.
----
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
