Skip to content

Conversation

@apivovarov
Copy link
Contributor

No description provided.

@shivaram
Copy link
Contributor

shivaram commented Dec 1, 2015

Jenkins, ok to test

@srowen
Copy link
Member

srowen commented Dec 1, 2015

Hm, I thought this script/support was removed from Spark just to avoid this kind of maintenance?

@shivaram
Copy link
Contributor

shivaram commented Dec 1, 2015

It hasn't been completely moved out yet -- We did the first part of the transition i.e. moving to amplab/spark-ec2 from mesos/spark-ec2, but the second part of moving this script out hasn't happened yet.

@SparkQA
Copy link

SparkQA commented Dec 1, 2015

Test build #46957 has finished for PR 10064 at commit ac2ac8e.

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

@shivaram
Copy link
Contributor

shivaram commented Dec 1, 2015

LGTM. Merging this. Thanks @apivovarov

asfgit pushed a commit that referenced this pull request Dec 1, 2015
Author: Alexander Pivovarov <apivovarov@gmail.com>

Closes #10064 from apivovarov/patch-1.
@JoshRosen
Copy link
Contributor

Hi @apivovarov, would you mind closing this PR now that it has been merged? GitHub can't auto-close PRs that are opened against backport branches. Thanks!

@apivovarov apivovarov closed this Dec 1, 2015
@apivovarov apivovarov deleted the patch-1 branch December 1, 2015 20:36
@apivovarov
Copy link
Contributor Author

We need to merge recent fixes from master to this branch #9975

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.

5 participants