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

[native] Enable additional Spark tests in CI #24368

Merged
merged 1 commit into from
Jan 16, 2025

Conversation

czentgr
Copy link
Contributor

@czentgr czentgr commented Jan 15, 2025

A few more tests are passing in the CI and are not flaky. The flaky tests are excluded and have issues opened.

Description

Motivation and Context

Impact

Test Plan

Contributor checklist

  • Please make sure your submission complies with our contributing guide, in particular code style and commit standards.
  • 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

Please follow release notes guidelines and fill in the release notes below.

== NO RELEASE NOTE ==

A few more tests are passing in the CI and are not flaky.
The flaky tests are excluded and have issues opened.
@prestodb-ci prestodb-ci added the from:IBM PR from IBM label Jan 15, 2025
@prestodb-ci prestodb-ci requested review from a team, wanglinsong and imsayari404 and removed request for a team January 15, 2025 15:08
@czentgr czentgr marked this pull request as ready for review January 15, 2025 18:04
@czentgr czentgr requested a review from a team as a code owner January 15, 2025 18:04
@czentgr
Copy link
Contributor Author

czentgr commented Jan 15, 2025

@majetideepak FYI, the original PR got merged before I could correct the list. It still contained tests that are flay. This list is excluding all the tests that have issues opened due to flakyness.

@tdcmeehan
Copy link
Contributor

FYI, I raised #24375 so these can be approved by C++ codeowners.

@tdcmeehan tdcmeehan merged commit 69229f2 into prestodb:master Jan 16, 2025
57 checks passed
Joe-Abraham pushed a commit to Joe-Abraham/presto that referenced this pull request Jan 20, 2025
A few more tests are passing in the CI and are not flaky.
The flaky tests are excluded and have issues opened.
shangm2 pushed a commit to shangm2/presto that referenced this pull request Jan 21, 2025
A few more tests are passing in the CI and are not flaky.
The flaky tests are excluded and have issues opened.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
from:IBM PR from IBM
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants