Skip to content

Conversation

@bernie-c
Copy link

Might have stepped on a couple of Matei's edits in Quick Start.

However please note that there doesn't seem to be an "addJar" method used in these code examples.

If "("local", "Simple App", "YOUR_SPARK_HOME",
List("target/scala-2.10/simple-project_2.10-1.0.jar")"
is a SparkConf object then maybe we need to make that clearer and put that part of Matei's language back in.

Matei wrote:

We pass the SparkContext constructor a
+SparkConf
+object which contains information about our
application. We also call sc.addJar to make sure that when our application is launched in cluster
mode, the jar file containing it will be shipped automatically to worker nodes.

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

pwendell pushed a commit to pwendell/spark that referenced this pull request May 12, 2014
Remove explicit conversion to PairRDDFunctions in cogroup()

As SparkContext._ is already imported, using the implicit conversion appears to make the code much cleaner. Perhaps there was some sinister reason for doing the conversion explicitly, however.

Author: Aaron Davidson <[email protected]>

== Merge branch commits ==

commit aa4a63f1bfd5b5178fe67364dd7ce4d84c357996
Author: Aaron Davidson <[email protected]>
Date:   Sun Feb 2 23:48:04 2014 -0800

    Remove explicit conversion to PairRDDFunctions in cogroup()

    As SparkContext._ is already imported, using the implicit conversion
    appears to make the code much cleaner. Perhaps there was some sinister
    reason for doing the converion explicitly, however.
@asfgit asfgit closed this in ee91eb8 Aug 27, 2014
bzhaoopenstack pushed a commit to bzhaoopenstack/spark that referenced this pull request Sep 11, 2019
arjunshroff pushed a commit to arjunshroff/spark that referenced this pull request Nov 24, 2020
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.

2 participants