Skip to content

Revert "Enhance join reordering to work with non-simple equi-join clauses"#20943

Merged
ajaygeorge merged 1 commit intoprestodb:masterfrom
aaneja:revert20413
Sep 22, 2023
Merged

Revert "Enhance join reordering to work with non-simple equi-join clauses"#20943
ajaygeorge merged 1 commit intoprestodb:masterfrom
aaneja:revert20413

Conversation

@aaneja
Copy link
Contributor

@aaneja aaneja commented Sep 22, 2023

This reverts commit 009c06b. from #20413

There is a subtle bug handling ProjectNodes between Join nodes that only do variable renames.
Reverting for now, while I figure out a better solution

@ajaygeorge
Copy link
Contributor

@aaneja Please remember to add a test case surfacing the bug when you add a fix for this. (ideally as a safety net even before the fix)

@ajaygeorge ajaygeorge merged commit 55cc36f into prestodb:master Sep 22, 2023
@wanglinsong wanglinsong mentioned this pull request Oct 3, 2023
54 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants