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

Fix the position of the jar path in the submit. #379

Closed

Conversation

andypetrella
Copy link
Contributor

This accordingly with the spark documentation: http://spark.apache.org/docs/latest/submitting-applications.html.

That is, the jar must come after the options like --master and so on, but mainly --class. Otherwise, no main can be found in the JAR.

This accordingly with the spark documentation: http://spark.apache.org/docs/latest/submitting-applications.html.

That is, the jar must come after the `options` like --master and so on, but mainly --class. Otherwise, no `main` can be found in the JAR.
@SparkQA
Copy link

SparkQA commented Sep 13, 2014

Can one of the admins verify this patch?

@fnothaft
Copy link
Member

Jenkins, add to whitelist.

@andypetrella
Copy link
Contributor Author

not going to work on jenkins due to port clash (need to change dynamically
the 4040 :-s)

aℕdy ℙetrella
about.me/noootsab
[image: aℕdy ℙetrella on about.me]

http://about.me/noootsab

On Sat, Sep 13, 2014 at 6:13 PM, Frank Austin Nothaft <
notifications@github.com> wrote:

Jenkins, add to whitelist.


Reply to this email directly or view it on GitHub
#379 (comment).

@fnothaft fnothaft added this to the 0.14.0 milestone Sep 13, 2014
@fnothaft
Copy link
Member

I think we'll want to merge this after #377.

@andypetrella
Copy link
Contributor Author

indeed

On Sat, Sep 13, 2014 at 11:34 PM, Frank Austin Nothaft <
notifications@github.com> wrote:

I think we'll want to merge this after #377
#377.


Reply to this email directly or view it on GitHub
#379 (comment).

@fnothaft
Copy link
Member

Committed manually (28b5da0). Thanks @andypetrella!

@fnothaft fnothaft closed this Sep 16, 2014
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