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.

Sign up for GitHub

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

Jump to bottom

[Build] merge changes to run-tests-jenkins from master branch #2237

Closed
wants to merge 2 commits into from
Closed

[Build] merge changes to run-tests-jenkins from master branch #2237

wants to merge 2 commits into from

Conversation

nchammas
Copy link
Contributor

@nchammas nchammas commented Sep 2, 2014

No description provided.

@nchammas nchammas mentioned this pull request Sep 2, 2014
Closed
@SparkQA
Copy link

SparkQA commented Sep 2, 2014

QA tests have started for PR 2237 at commit 39bdd5e.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 2, 2014

QA tests have finished for PR 2237 at commit 39bdd5e.

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

@pwendell
Copy link
Contributor

pwendell commented Sep 2, 2014

thanks @nchammas I'll pull this in.

asfgit pushed a commit that referenced this pull request Sep 2, 2014
@nchammas @pwendell
[Build] merge changes to run-tests-jenkins from master branch
dff42a7 
Author: Nicholas Chammas <nicholas.chammas@gmail.com>
Author: nchammas <nicholas.chammas@gmail.com>

Closes #2237 from nchammas/branch-1.1 and squashes the following commits:

39bdd5e [Nicholas Chammas] merge updates from master
f5aa841 [nchammas] Merge pull request #3 from apache/branch-1.1
@nchammas
Copy link
Contributor Author

nchammas commented Sep 2, 2014

Hey @pwendell, looks like asfgit didn't close this PR when you merged in the changes. Is that expected? Shall I close this PR myself?

@JoshRosen
Copy link
Contributor

@nchammas Yeah, please go ahead and close it. GitHub only auto-closes PRs once the commit makes it to the master branch, which won't happen here.

@nchammas nchammas closed this Sep 2, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@nchammas @SparkQA @pwendell @JoshRosen