Skip to content

Conversation

@felixcheung
Copy link
Contributor

Include info on SPARK_YARN_JAR can be HDFS

@felixcheung
Copy link
Contributor Author

@jongyoul @swkimme Could you please review?

@jongyoul
Copy link
Contributor

LGTM. Thanks for fix my typo.

@swkimme
Copy link
Contributor

swkimme commented Mar 25, 2015

Looks great!

swkimme added a commit that referenced this pull request Mar 25, 2015
minor doc update for running on YARN
@swkimme swkimme merged commit dc46467 into ZEPL:master Mar 25, 2015
@jongyoul
Copy link
Contributor

@swkimme It is ok to merge this PR? I've heard Zeppelin repos moved to apache/incubator-zeppelin. I think @felixcheung should make this PR with same patch under apache/incubator-zeppelin. How do you think of it?

@swkimme
Copy link
Contributor

swkimme commented Mar 25, 2015

@jongyoul
Oh, you're right.
I guess we need to create this PR again for apache/incubator-zeppelin repo.
@felixcheung can you do that?

@felixcheung
Copy link
Contributor Author

Absolutely! Created apache/zeppelin#2

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