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

[SPARK-3802][BUILD] Scala version is wrong in dev/audit-release/blank_sbt_build/build.sbt #2661

Closed
wants to merge 4 commits into from

Conversation

sarutak
Copy link
Member

@sarutak sarutak commented Oct 5, 2014

In dev/audit-release/blank_sbt_build/build.sbt, scalaVersion indicates 2.9.3 but I think 2.10.4 is correct.

@SparkQA
Copy link

SparkQA commented Oct 5, 2014

QA tests have started for PR 2661 at commit 58a9666.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Oct 5, 2014

QA tests have finished for PR 2661 at commit 58a9666.

  • This patch passes unit 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/21312/Test PASSed.

@SparkQA
Copy link

SparkQA commented Oct 6, 2014

QA tests have started for PR 2661 at commit 7090e17.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Oct 6, 2014

QA tests have finished for PR 2661 at commit 7090e17.

  • This patch passes unit 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/21326/Test PASSed.

@AmplabJenkins
Copy link

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

@sarutak
Copy link
Member Author

sarutak commented Oct 7, 2014

retest this please.

@AmplabJenkins
Copy link

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

@sarutak
Copy link
Member Author

sarutak commented Oct 7, 2014

retest this please.

@SparkQA
Copy link

SparkQA commented Oct 7, 2014

QA tests have started for PR 2661 at commit 8b64bb7.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Oct 7, 2014

QA tests have finished for PR 2661 at commit 8b64bb7.

  • 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/21383/Test PASSed.

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@nchammas
Copy link
Contributor

nchammas commented Dec 8, 2014

@sarutak It looks like this code has now been changed to read from an environment variable. Do we still need this PR?

@JoshRosen
Copy link
Contributor

Yeah, it looks like this has been subsumed by Andrew's changes in 723a86b, so do you mind closing this PR? Thanks!

@sarutak
Copy link
Member Author

sarutak commented Dec 9, 2014

O.K, I'll close this PR.

@sarutak sarutak closed this Dec 9, 2014
@sarutak sarutak deleted the SPARK-3802 branch April 11, 2015 05:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants