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.

Sign up for GitHub

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

Jump to bottom

Add rule to push filter through count() aggregation #12047

Merged
merged 1 commit into from
Apr 21, 2022
Merged

Add rule to push filter through count() aggregation #12047

merged 1 commit into from
Apr 21, 2022

Conversation

kasiafi
Copy link
Member

@kasiafi kasiafi commented Apr 20, 2022 •
edited
Loading

Is this change a fix, improvement, new feature, refactoring, or other?
optimization

Release notes

# General
* Improve performance of queries involving correlated IN or EXISTS predicate. ({issue}`12047`)

@cla-bot cla-bot bot added the cla-signed label Apr 20, 2022
@findepi findepi requested a review from sopel39 April 20, 2022 10:04
@sopel39 sopel39 requested a review from raunaqmorarka April 20, 2022 10:07
@sopel39
Copy link
Member

sopel39 commented Apr 20, 2022

@kasiafi do you know the perf diff?

@sopel39 sopel39 merged commit 0b823cc into trinodb:master Apr 21, 2022
@sopel39 sopel39 mentioned this pull request Apr 21, 2022
Closed
@github-actions github-actions bot added this to the 378 milestone Apr 21, 2022
@mosabua mosabua mentioned this pull request Apr 21, 2022
Merged
@takezoe takezoe mentioned this pull request Nov 3, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sopel39 sopel39 sopel39 approved these changes

@raunaqmorarka raunaqmorarka Awaiting requested review from raunaqmorarka

Assignees
No one assigned
Labels
cla-signed
Milestone
378
Development

Successfully merging this pull request may close these issues.
2 participants
@kasiafi @sopel39