Skip to content

SPARK-4767: Add support for launching in a specified placement group to spark_ec2#3623

Closed
holdenk wants to merge 2 commits intoapache:masterfrom
holdenk:SPARK-4767-add-support-for-launching-in-a-specified-placement-group-to-spark-ec2-scripts
Closed

SPARK-4767: Add support for launching in a specified placement group to spark_ec2#3623
holdenk wants to merge 2 commits intoapache:masterfrom
holdenk:SPARK-4767-add-support-for-launching-in-a-specified-placement-group-to-spark-ec2-scripts

Conversation

@holdenk
Copy link
Contributor

@holdenk holdenk commented Dec 5, 2014

Placement groups are cool and all the cool kids are using them. Lets add support for them to spark_ec2.py because I'm lazy

…add support for them to spark_ec2.py because I'm lazy
@SparkQA
Copy link

SparkQA commented Dec 5, 2014

Test build #24190 has started for PR 3623 at commit 70ace25.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Dec 5, 2014

Test build #24190 has finished for PR 3623 at commit 70ace25.

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

@SparkQA
Copy link

SparkQA commented Dec 16, 2014

Test build #24506 has started for PR 3623 at commit 111a5fd.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Dec 16, 2014

Test build #24506 has finished for PR 3623 at commit 111a5fd.

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

@JoshRosen
Copy link
Contributor

This LGTM, so I'm merging this into master. Thanks!

@asfgit asfgit closed this in b0dfdbd Dec 16, 2014
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