Skip to content

Conversation

@rxin
Copy link
Contributor

@rxin rxin commented Sep 27, 2014

This addresses some minor issues in #2425

@AmplabJenkins
Copy link

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/20905/

@SparkQA
Copy link

SparkQA commented Sep 27, 2014

QA tests have started for PR 2557 at commit a51e5f6.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 27, 2014

QA tests have finished for PR 2557 at commit a51e5f6.

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

@rxin
Copy link
Contributor Author

rxin commented Sep 27, 2014

hm don't think the test error is related.

@SparkQA
Copy link

SparkQA commented Sep 27, 2014

QA tests have started for PR 2557 at commit a51e5f6.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 27, 2014

QA tests have finished for PR 2557 at commit a51e5f6.

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

@SparkQA
Copy link

SparkQA commented Sep 27, 2014

QA tests have started for PR 2557 at commit a51e5f6.

  • This patch does not merge cleanly!

@SparkQA
Copy link

SparkQA commented Sep 27, 2014

QA tests have finished for PR 2557 at commit a51e5f6.

  • This patch passes unit tests.
  • This patch does not merge cleanly!

@rxin
Copy link
Contributor Author

rxin commented Sep 27, 2014

@nchammas any idea why it says does not merge cleanly even though it does?

@rxin
Copy link
Contributor Author

rxin commented Sep 27, 2014

Ok merging this now.

@asfgit asfgit closed this in 5b922bb Sep 27, 2014
@nchammas
Copy link
Contributor

@rxin The block that sets this message is driven by an environment variable set outside the scope of the Jenkins script in this repo, so that must be mis-set somehow.

Perhaps it's related to some work @JoshRosen said he was going to be doing with the AMPLab team this weekend to fix the double-posting of test result messages to GitHub?

@rxin rxin deleted the TaskContext branch September 27, 2014 22:12
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