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

[EC2] Update default Spark version to 1.2.0 #3793

Closed
wants to merge 2 commits into from

Conversation

nchammas
Copy link
Contributor

Now that 1.2.0 is out, let's update the default Spark version.

@nchammas
Copy link
Contributor Author

cc @JoshRosen

@SparkQA
Copy link

SparkQA commented Dec 24, 2014

Test build #24787 has started for PR 3793 at commit ec0e904.

  • This patch merges cleanly.

@JoshRosen
Copy link
Contributor

It looks like this was already done in branch-1.2, but it doesn't hurt to do it in master: dfb8c65

If it's not already there, we should add this to the "preparing a release" checklist on the wiki.

@nchammas
Copy link
Contributor Author

Hmm, master is already on 1.3.0 in that config file in dfb8c65.

@SparkQA
Copy link

SparkQA commented Dec 24, 2014

Test build #24787 has finished for PR 3793 at commit ec0e904.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@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/24787/
Test PASSed.

@JoshRosen
Copy link
Contributor

That other config change is for documentation generation; we can't bump the EC2 version to a snapshot release. This looks good to me.

@nchammas
Copy link
Contributor Author

Oh, hold up, we need to update this map, too...

@SparkQA
Copy link

SparkQA commented Dec 24, 2014

Test build #24790 has started for PR 3793 at commit 3255832.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Dec 24, 2014

Test build #24790 has finished for PR 3793 at commit 3255832.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@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/24790/
Test PASSed.

@@ -255,6 +255,7 @@ def get_spark_shark_version(opts):
"1.0.1": "1.0.1",
"1.0.2": "1.0.2",
"1.1.0": "1.1.0",
"1.2.0": "1.2.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

Probably also want to have 1.1.1 in here since it's also in branch-1.2; I'll just add this myself on merge, though.

@JoshRosen
Copy link
Contributor

LGTM, so I'll merge this into master (1.3.0).

@asfgit asfgit closed this in b6b6393 Dec 25, 2014
@nchammas nchammas deleted the patch-1 branch December 25, 2014 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants