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

Skip testStdDev in SingleStore tests #22565

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Skip testStdDev in SingleStore tests #22565

wants to merge 1 commit into from

Conversation

elharo
Copy link
Contributor

@elharo elharo commented Apr 19, 2024

Description

Ignore this test in singlestore until the root cause is fixed.

Motivation and Context

Deflake

Impact

fixes #22564

Test Plan

CI

Contributor checklist

  • Please make sure your submission complies with our development, formatting, commit message, and attribution guidelines.
  • PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
  • Documented new properties (with its default value), SQL syntax, functions, or other functionality.
  • If release notes are required, they follow the release notes guidelines.
  • Adequate tests were added if applicable.
  • CI passed.

Release Notes

== NO RELEASE NOTE ==

@elharo elharo requested a review from tdcmeehan April 19, 2024 15:29
@elharo elharo marked this pull request as ready for review April 19, 2024 15:29
@elharo elharo requested a review from a team as a code owner April 19, 2024 15:29
@elharo elharo requested a review from presto-oss April 19, 2024 15:29
@elharo elharo changed the title Skip testStdDev Skip testStdDev in SingleStore tests Apr 19, 2024
@tdcmeehan
Copy link
Contributor

Do you have any idea why this specific test is effected? And have you seen this problem in more than one PR?

@elharo
Copy link
Contributor Author

elharo commented Apr 23, 2024

I've seen it more than once. Seems to be a bug in singlestore, not in presto

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.

Flaky TestSingleStoreDistributedQueries.testStdDev
2 participants