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-10086] [MLlib] [Streaming] [PySpark] ignore StreamingKMeans test in PySpark for now #10909

Closed
wants to merge 1 commit into from

Conversation

mengxr
Copy link
Contributor

@mengxr mengxr commented Jan 26, 2016

I saw several failures from recent PR builds, e.g., https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50015/consoleFull. This PR marks the test as ignored and we will fix the flakyness in SPARK-10086.

@gliptak Do you know why the test failure didn't show up in the Jenkins "Test Result"?

cc: @jkbradley

@mengxr mengxr changed the title [MLlib] [Streaming] [PySpark] ignore StreamingKMeans test in PySpark for now [SPARK-10086] [MLlib] [Streaming] [PySpark] ignore StreamingKMeans test in PySpark for now Jan 26, 2016
@SparkQA
Copy link

SparkQA commented Jan 26, 2016

Test build #50049 has finished for PR 10909 at commit d7e7aa9.

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

@gliptak
Copy link
Contributor

gliptak commented Jan 26, 2016

@mengxr Does the test only fail under one of the Python versions?

@jkbradley
Copy link
Member

@mengxr Were these all the same test failing? Before, I had seen similar failures from multiple tests.
Also, did the failure rate seem to increase recently?

@jkbradley
Copy link
Member

LGTM though, if you want to go ahead and disable it

@mengxr
Copy link
Contributor Author

mengxr commented Jan 26, 2016

Some (not all) failures in the last 4 days:

@gliptak The builds listed above have XML report, so I can track test failures. However, the one I posted in PR description (build 50015) doesn't report test result. The test failed with Python 3.4.

Merged into master.

@asfgit asfgit closed this in 27c910f Jan 26, 2016
@maver1ck
Copy link
Contributor

maver1ck commented Jul 28, 2016

@jkbradley
What about merging this to branch-1.6?
I have same problem here:
#14390

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.

5 participants