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

fix(cubesql): Use pushdown-pullup scheme for FilterSimplifyReplacer #9278

Draft

mcheshkov wants to merge 2 commits into master from filter-replacer-push-pull

Draft

fix(cubesql): Use pushdown-pullup scheme for FilterSimplifyReplacer #9278

[WIP] fix(cubesql): Use pushdown-pullup scheme for FilterSimplifyRepl…

Codecov / codecov/patch succeeded Feb 26, 2025 in 1s

97.02% of diff hit (target 83.65%)

View this Pull Request on Codecov

97.02% of diff hit (target 83.65%)

Annotations

Check warning on line 2620 in rust/cubesql/cubesql/src/compile/rewrite/rules/filters.rs

codecov / codecov/patch

rust/cubesql/cubesql/src/compile/rewrite/rules/filters.rs#L2620

Added line #L2620 was not covered by tests

Check warning on line 2623 in rust/cubesql/cubesql/src/compile/rewrite/rules/filters.rs

codecov / codecov/patch

rust/cubesql/cubesql/src/compile/rewrite/rules/filters.rs#L2622-L2623

Added lines #L2622 - L2623 were not covered by tests

Check warning on line 2633 in rust/cubesql/cubesql/src/compile/rewrite/rules/filters.rs

codecov / codecov/patch

rust/cubesql/cubesql/src/compile/rewrite/rules/filters.rs#L2626-L2633

Added lines #L2626 - L2633 were not covered by tests

Check warning on line 2715 in rust/cubesql/cubesql/src/compile/rewrite/rules/filters.rs

codecov / codecov/patch

rust/cubesql/cubesql/src/compile/rewrite/rules/filters.rs#L2715

Added line #L2715 was not covered by tests

View more details on Codecov