Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update run-example #988

Closed
wants to merge 1 commit into from
Closed

Update run-example #988

wants to merge 1 commit into from

Conversation

maji2014
Copy link
Contributor

@maji2014 maji2014 commented Jun 6, 2014

Old code can only be ran under spark_home and use "bin/run-example".
Error "./run-example: line 55: ./bin/spark-submit: No such file or directory" appears when running in other place. So change this

Old code can only be ran under spark_home and use "bin/run-example".
Error "./run-example: line 55: ./bin/spark-submit: No such file or directory" appears when running in other place. So change this
@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@maji2014
Copy link
Contributor Author

maji2014 commented Jun 6, 2014

Maybe it's better to change it to "$FWDIR"/bin/spark-submit and commit it into apache:master

@pwendell
Copy link
Contributor

pwendell commented Jun 6, 2014

Thanks for finding this fix! Since this is a user-facing bug, do you mind creating a JIRA? Also it is slightly better if you make the PR into the master branch. And also yes, adding quotes would better too.

@maji2014
Copy link
Contributor Author

maji2014 commented Jun 6, 2014

OK, SPARK-2057 is opened for tracing this issue.

@pwendell
Copy link
Contributor

pwendell commented Jun 8, 2014

Thanks for creating the JIRA! Would you mind (a) adding the quotes around the dir and (b) opening this against the master branch? Then we can merge it and back-port it into 1.0.1

@@ -51,7 +51,7 @@ if [[ ! $EXAMPLE_CLASS == org.apache.spark.examples* ]]; then
EXAMPLE_CLASS="org.apache.spark.examples.$EXAMPLE_CLASS"
fi

./bin/spark-submit \
$FWDIR/bin/spark-submit \
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be better to do:

"$FWDIR"/bin/spark-submit

@maji2014
Copy link
Contributor Author

maji2014 commented Jun 8, 2014

i agree that and patch #1011 is opened against the master branch. you can merge it and back-port it into 1.0.1

@pwendell pwendell mentioned this pull request Jun 8, 2014
@asfgit asfgit closed this in e9261d0 Jun 8, 2014
asfgit pushed a commit that referenced this pull request Jun 8, 2014
Old code can only be ran under spark_home and use "bin/run-example".
 Error "./run-example: line 55: ./bin/spark-submit: No such file or directory" appears when running in other place. So change this

Author: maji2014 <maji3@asiainfo-linkage.com>

Closes #1011 from maji2014/master and squashes the following commits:

2cc1af6 [maji2014] Update run-example

Closes #988.
(cherry picked from commit e9261d0)

Signed-off-by: Patrick Wendell <pwendell@gmail.com>
pdeyhim pushed a commit to pdeyhim/spark-1 that referenced this pull request Jun 25, 2014
Old code can only be ran under spark_home and use "bin/run-example".
 Error "./run-example: line 55: ./bin/spark-submit: No such file or directory" appears when running in other place. So change this

Author: maji2014 <maji3@asiainfo-linkage.com>

Closes apache#1011 from maji2014/master and squashes the following commits:

2cc1af6 [maji2014] Update run-example

Closes apache#988.
@maji2014 maji2014 deleted the branch-1.0 branch July 19, 2014 09:24
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
Old code can only be ran under spark_home and use "bin/run-example".
 Error "./run-example: line 55: ./bin/spark-submit: No such file or directory" appears when running in other place. So change this

Author: maji2014 <maji3@asiainfo-linkage.com>

Closes apache#1011 from maji2014/master and squashes the following commits:

2cc1af6 [maji2014] Update run-example

Closes apache#988.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants