Skip to content

Conversation

@andrewor14
Copy link
Contributor

Simple omission on my part.

@SparkQA
Copy link

SparkQA commented Dec 5, 2014

Test build #24153 has started for PR 3612 at commit 1f03b60.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Dec 5, 2014

Test build #24153 has finished for PR 3612 at commit 1f03b60.

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

@JoshRosen
Copy link
Contributor

LGTM. I'm going to merge this into master and add a backport-needed tag so we don't forget to merge this into branch-1.2.

@asfgit asfgit closed this in 65f929d Dec 9, 2014
@andrewor14 andrewor14 deleted the dynamic-allocation-synchronization branch December 9, 2014 20:25
asfgit pushed a commit that referenced this pull request Dec 17, 2014
Simple omission on my part.

Author: Andrew Or <[email protected]>

Closes #3612 from andrewor14/dynamic-allocation-synchronization and squashes the following commits:

1f03b60 [Andrew Or] Synchronize kills

(cherry picked from commit 65f929d)
Signed-off-by: Josh Rosen <[email protected]>
@JoshRosen
Copy link
Contributor

I've backported this to branch-1.2.

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