Skip to content

Ignore sort_query_fuzzer_runner #16462

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

Merged
merged 1 commit into from
Jun 20, 2025
Merged

Conversation

blaginin
Copy link
Contributor

Rationale for this change

See #16452

Test keep keeps failing. I think it's important to keep CI green, so let's skip it now

What changes are included in this PR?

Skip the test. Will be unskipped when #16452 is solved

Are these changes tested?

Are there any user-facing changes?

@blaginin blaginin self-assigned this Jun 19, 2025
@github-actions github-actions bot added the core Core DataFusion crate label Jun 19, 2025
Copy link
Contributor

@2010YOUY01 2010YOUY01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you. Let's disable it for now and re-enable it after the issue is resolved.

@2010YOUY01 2010YOUY01 merged commit aa1e6da into apache:main Jun 20, 2025
27 checks passed
@ozankabak
Copy link
Contributor

Isn't this fixed by #16465?

@2010YOUY01
Copy link
Contributor

Isn't this fixed by #16465?

Ah yes, and thank you for the pointer.

#16465 suppressed the specific failing test case, so now we should be able to let the fuzzer run in CI.

2010YOUY01 added a commit that referenced this pull request Jun 20, 2025
alamb pushed a commit that referenced this pull request Jun 20, 2025
@alamb
Copy link
Contributor

alamb commented Jun 20, 2025

🤔 the test seems to have failed again:

So we should revert the revert

alamb added a commit to alamb/datafusion that referenced this pull request Jun 20, 2025
alamb added a commit that referenced this pull request Jun 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Core DataFusion crate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants