-
Notifications
You must be signed in to change notification settings - Fork 28.6k
[SPARK-31732][TESTS] Disable some flaky tests temporarily #28547
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
Conversation
Hi, @cloud-fan . So, you are targeting this PR only in BTW, the most annoying failures came from |
Do you know how frequently it fail? In master Jenkins build, the following failures occur, too.
|
Test build #122684 has finished for PR 28547 at commit
|
In another PR (#28509 (comment)),
|
@dongjoon-hyun, actually some of them are not related to the corrupt local repository. This was in amp-jenkins-worker-03 I am retriggering the builds for now. Let's disable them and run the RC if they are still frequently failed. |
@dongjoon-hyun I'll disable these tests as well, please let me know if you see more. |
Thanks. @cloud-fan . I'm +1 for disabling the above, too. Yes, @HyukjinKwon . The above are really flaky tests. What I meant was |
retest this please |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Test build #122710 has finished for PR 28547 at commit
|
Test build #122718 has finished for PR 28547 at commit
|
retest this please |
Test build #122722 has finished for PR 28547 at commit
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1, LGTM. Thank you all.
Merged to master/3.0.
### What changes were proposed in this pull request? It's quite annoying to be blocked by flaky tests in several PRs. This PR disables them. The tests come from 3 PRs I'm recently watching: #28526 #28463 #28517 ### Why are the changes needed? To make PR builder more stable ### Does this PR introduce _any_ user-facing change? no ### How was this patch tested? N/A Closes #28547 from cloud-fan/test. Authored-by: Wenchen Fan <wenchen@databricks.com> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org> (cherry picked from commit 2012d58) Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
cc @rxin since he is a release manager of 3.0.0. |
What changes were proposed in this pull request?
It's quite annoying to be blocked by flaky tests in several PRs. This PR disables them. The tests come from 3 PRs I'm recently watching:
#28526
#28463
#28517
Why are the changes needed?
To make PR builder more stable
Does this PR introduce any user-facing change?
no
How was this patch tested?
N/A