Skip to content

Conversation

@jerryshao
Copy link
Contributor

I think this issue is introduced in #5732

@AmplabJenkins
Copy link

Merged build triggered.

@AmplabJenkins
Copy link

Merged build started.

@SparkQA
Copy link

SparkQA commented May 6, 2015

Test build #31966 has started for PR 5937 at commit 4206ee0.

@SparkQA
Copy link

SparkQA commented May 6, 2015

Test build #31966 has finished for PR 5937 at commit 4206ee0.

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

@AmplabJenkins
Copy link

Merged build finished. Test FAILed.

@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/31966/
Test FAILed.

@jerryshao
Copy link
Contributor Author

Jenkins, retest this please.

@AmplabJenkins
Copy link

Merged build triggered.

@AmplabJenkins
Copy link

Merged build started.

@SparkQA
Copy link

SparkQA commented May 6, 2015

Test build #31974 has started for PR 5937 at commit 4206ee0.

@SparkQA
Copy link

SparkQA commented May 6, 2015

Test build #31974 has finished for PR 5937 at commit 4206ee0.

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

@AmplabJenkins
Copy link

Merged build finished. Test PASSed.

@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/31974/
Test PASSed.

@tdas
Copy link
Contributor

tdas commented May 7, 2015

I am choosing #5950 in favor of this one as that one has a test to prevent this from happening in the future.Thanks though for catching this! Mind closing this for now.

@jerryshao
Copy link
Contributor Author

Hi @tdas, please take a look at this PR, I think this is a problem introduced by #5732 , and should be fixed. Thanks a lot :)

@jerryshao
Copy link
Contributor Author

OK, cool.

@jerryshao jerryshao closed this May 7, 2015
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