Skip to content
This repository was archived by the owner on Jan 7, 2025. It is now read-only.

feat: Eliminate Filter Rule #54

Merged
merged 11 commits into from
Feb 12, 2024
Merged

feat: Eliminate Filter Rule #54

merged 11 commits into from
Feb 12, 2024

Conversation

jurplel
Copy link
Member

@jurplel jurplel commented Feb 10, 2024

TODO: Write documentation & write tests

Depends on #52

@jurplel jurplel marked this pull request as draft February 10, 2024 07:01
@jurplel jurplel changed the title Eliminate Filter Rule feat: Eliminate Join Rule Feb 10, 2024
@jurplel jurplel mentioned this pull request Feb 10, 2024
26 tasks
@yliang412
Copy link
Member

I think this is eliminate filter rule right?

@jurplel jurplel changed the title feat: Eliminate Join Rule feat: Eliminate Filter Rule Feb 10, 2024
@jurplel
Copy link
Member Author

jurplel commented Feb 10, 2024

Yes, thanks for the correction

@jurplel jurplel marked this pull request as ready for review February 11, 2024 22:01
@jurplel jurplel requested a review from yliang412 February 11, 2024 23:38
Copy link
Member

@yliang412 yliang412 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 for the code comments!

@jurplel jurplel merged commit cfa595b into main Feb 12, 2024
1 check passed
@jurplel jurplel deleted the bowad/eliminate-filter branch February 12, 2024 22:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants