Skip to content

[CALCITE-6873] FilterProjectTransposeRule should not push the Filter …#4226

Merged
NobiGo merged 1 commit into
apache:mainfrom
NobiGo:CALCITE-6873
Mar 7, 2025
Merged

[CALCITE-6873] FilterProjectTransposeRule should not push the Filter …#4226
NobiGo merged 1 commit into
apache:mainfrom
NobiGo:CALCITE-6873

Conversation

@NobiGo
Copy link
Copy Markdown
Contributor

@NobiGo NobiGo commented Mar 5, 2025

…past the Project when the Filter contains a Subquery with correlation

Comment thread core/src/main/java/org/apache/calcite/rel/rules/FilterProjectTransposeRule.java Outdated
Comment thread core/src/main/java/org/apache/calcite/rex/RexUtil.java
@NobiGo NobiGo added the LGTM-will-merge-soon Overall PR looks OK. Only minor things left. label Mar 6, 2025
Copy link
Copy Markdown
Member

@asolimando asolimando left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @NobiGo!

…past the Project when the Filter contains a Subquery with correlation
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Mar 7, 2025

@NobiGo NobiGo merged commit 802fce3 into apache:main Mar 7, 2025
@NobiGo NobiGo deleted the CALCITE-6873 branch March 8, 2025 01:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

LGTM-will-merge-soon Overall PR looks OK. Only minor things left.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants