Skip to content

Extended tests failing on main #15359

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

Closed
alamb opened this issue Mar 22, 2025 · 3 comments · Fixed by #15368
Closed

Extended tests failing on main #15359

alamb opened this issue Mar 22, 2025 · 3 comments · Fixed by #15368
Labels
bug Something isn't working

Comments

@alamb
Copy link
Contributor

alamb commented Mar 22, 2025

Describe the bug

example

https://github.com/apache/datafusion/actions/runs/14005503215/job/39218865563

External error: query is expected to fail with error:
	(regex) DataFusion error: Error during planning: Projection references non\-aggregate values: Expression cor0\.col1 could not be resolved from available columns: cor0\.col2
but got error:
	DataFusion error: Error during planning: Column in SELECT must be in GROUP BY or an aggregate function: While expanding wildcard, column "cor0.col1" must appear in the GROUP BY clause or must be part of an aggregate function, currently only "cor0.col2" appears in the SELECT clause satisfies this requirement
[SQL] SELECT - COALESCE ( - 54, + cor0.col1 * + NULLIF ( + cor0.col1, cor0.col0 ), - cor0.col1 ) AS col0 FROM tab1 cor0 GROUP BY cor0.col2
at ../../datafusion-testing/data/sqlite/random/groupby/slt_good_7.slt:14159

Happened first with this PR:

I think the expected messages just need updating

To Reproduce

No response

Expected behavior

No response

Additional context

No response

@alamb alamb added the bug Something isn't working label Mar 22, 2025
@alamb
Copy link
Contributor Author

alamb commented Mar 22, 2025

@goldmedal
Copy link
Contributor

I created apache/datafusion-testing#8 for this issue.

@alamb
Copy link
Contributor Author

alamb commented Mar 23, 2025

I created apache/datafusion-testing#8 for this issue.

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants