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

fix(cubesql): Avoid panics during filter rewrites #9166

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
+115 −20

fix(cubesql): Avoid panics during filter rewrites

910b324
Select commit
Loading
Failed to load commit list.
Open

fix(cubesql): Avoid panics during filter rewrites #9166

fix(cubesql): Avoid panics during filter rewrites
910b324
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Jan 31, 2025 in 1s

89.79% of diff hit (target 83.17%)

View this Pull Request on Codecov

89.79% of diff hit (target 83.17%)

Annotations

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L3563 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

rust/cubesql/cubesql/src/compile/rewrite/rules/filters.rs#L3577-L3578 

Added lines #L3577 - L3578 were not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

rust/cubesql/cubesql/src/compile/rewrite/rules/filters.rs#L3605-L3606 

Added lines #L3605 - L3606 were not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

rust/cubesql/cubesql/src/compile/rewrite/rules/filters.rs#L3611-L3613 

Added lines #L3611 - L3613 were not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L3887 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L3890 was not covered by tests
View more details on Codecov