Skip to content

[SPARK-7092] Update spark scala version to 2.11.6 #5662

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

Closed

Conversation

ScrapCodes
Copy link
Member

No description provided.

@ScrapCodes
Copy link
Member Author

@pwendell Please take a look :)

@SparkQA
Copy link

SparkQA commented Apr 23, 2015

Test build #30838 has started for PR 5662 at commit 58cf4f9.

@@ -1129,7 +1129,7 @@ class SparkIMain(@BeanProperty val factory: ScriptEngineFactory, initialSettings

def apply(line: String): Result = debugging(s"""parse("$line")""") {
var isIncomplete = false
currentRun.reporting.withIncompleteHandler((_, _) => isIncomplete = true) {
currentRun.parsing.withIncompleteHandler((_, _) => isIncomplete = true) {
Copy link
Member

Choose a reason for hiding this comment

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

Just curious, was this a break across maintenance releases? or is this basically not a public Scala API anyway?

Are there any potential gotchas in updating? You know more than I would. I tend to favor updating as well to get fixes.

Copy link
Member Author

Choose a reason for hiding this comment

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

This was the change, it corresponds to scala/scala@64ebac2

Copy link
Member Author

Choose a reason for hiding this comment

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

It is harmless, if not beneficial. But this can be a stepping stone towards enabling add jars, because then we have two options 1) back port scala's version of add jars 2) port spark scala 2.10 repl's version of add jars on the fly.

Without this patch such a option does not exist. I agree best thing is to patch scala/scala so that we do not have to maintain repl(s). So I am working on it, with whatever time I have.

@SparkQA
Copy link

SparkQA commented Apr 23, 2015

Test build #30838 has finished for PR 5662 at commit 58cf4f9.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.
  • This patch does not change any dependencies.

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30838/
Test PASSed.

@asfgit asfgit closed this in a11c868 Apr 25, 2015
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request May 14, 2015
Author: Prashant Sharma <prashant.s@imaginea.com>

Closes apache#5662 from ScrapCodes/SPARK-7092/scala-update-2.11.6 and squashes the following commits:

58cf4f9 [Prashant Sharma] [SPARK-7092] Update spark scala version to 2.11.6
@ScrapCodes ScrapCodes deleted the SPARK-7092/scala-update-2.11.6 branch June 3, 2015 05:57
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
Author: Prashant Sharma <prashant.s@imaginea.com>

Closes apache#5662 from ScrapCodes/SPARK-7092/scala-update-2.11.6 and squashes the following commits:

58cf4f9 [Prashant Sharma] [SPARK-7092] Update spark scala version to 2.11.6
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.

4 participants