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-4264] Completion iterator should only invoke callback once #3128

Closed
wants to merge 1 commit into from

Conversation

aarondav
Copy link
Contributor

@aarondav aarondav commented Nov 6, 2014

No description provided.

@aarondav
Copy link
Contributor Author

aarondav commented Nov 6, 2014

@marmbrus

@aarondav
Copy link
Contributor Author

aarondav commented Nov 6, 2014

This should be pushed to 1.2.0 as well.

@SparkQA
Copy link

SparkQA commented Nov 6, 2014

Test build #22979 has started for PR 3128 at commit 698e4be.

  • This patch merges cleanly.

@rxin
Copy link
Contributor

rxin commented Nov 6, 2014

lgtm

@SparkQA
Copy link

SparkQA commented Nov 6, 2014

Test build #22979 has finished for PR 3128 at commit 698e4be.

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

@aarondav
Copy link
Contributor Author

aarondav commented Nov 6, 2014

Merging this into master and branch-1.2.

@asfgit asfgit closed this in 23eaf0e Nov 6, 2014
asfgit pushed a commit that referenced this pull request Nov 6, 2014
Author: Aaron Davidson <aaron@databricks.com>

Closes #3128 from aarondav/compiter and squashes the following commits:

698e4be [Aaron Davidson] [SPARK-4264] Completion iterator should only invoke callback once

(cherry picked from commit 23eaf0e)
Signed-off-by: Aaron Davidson <aaron@databricks.com>
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