Skip to content

Add tests for simplifying multiple aggregate expressions#20723

Merged
alamb merged 1 commit intoapache:mainfrom
alamb:alamb/just_tests
Mar 9, 2026
Merged

Add tests for simplifying multiple aggregate expressions#20723
alamb merged 1 commit intoapache:mainfrom
alamb:alamb/just_tests

Conversation

@alamb
Copy link
Contributor

@alamb alamb commented Mar 5, 2026

Which issue does this PR close?

Rationale for this change

As part of #15524 I am working on some optimizations for queries with multiple aggregates. To make it clear what is changing, and ensure I don't introduce regressions, I want to add the tests to main first.

Merging the tests first should also make #20749 easier to review

What changes are included in this PR?

  1. Add new tests

Are these changes tested?

Yes, all tests

Are there any user-facing changes?

No

@github-actions github-actions bot added the sqllogictest SQL Logic Tests (.slt) label Mar 5, 2026
Copy link
Contributor

@adriangb adriangb left a comment

Choose a reason for hiding this comment

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

let's 🚢 it!

@alamb alamb added this pull request to the merge queue Mar 9, 2026
@alamb
Copy link
Contributor Author

alamb commented Mar 9, 2026

Thanks @adriangb

Merged via the queue into apache:main with commit bb421db Mar 9, 2026
31 of 32 checks passed
@alamb alamb deleted the alamb/just_tests branch March 9, 2026 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sqllogictest SQL Logic Tests (.slt)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants