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

Update AggregateRel.CopyWithExpressionRewrite to rewrite measure functions in addition to filters #112

Merged
merged 8 commits into from
Jan 29, 2025

more lint fixes

d36bac9
Select commit
Loading
Failed to load commit list.
Merged

Update AggregateRel.CopyWithExpressionRewrite to rewrite measure functions in addition to filters #112

more lint fixes
d36bac9
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Jan 29, 2025 in 0s

36.58% of diff hit (target 63.64%)

View this Pull Request on Codecov

36.58% of diff hit (target 63.64%)

Annotations

Check warning on line 828 in expr/functions.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

expr/functions.go#L814-L828 

Added lines #L814 - L828 were not covered by tests

Check warning on line 833 in expr/functions.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

expr/functions.go#L832-L833 

Added lines #L832 - L833 were not covered by tests

Check warning on line 1174 in plan/relations.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

plan/relations.go#L1173-L1174 

Added lines #L1173 - L1174 were not covered by tests

Check warning on line 1184 in plan/relations.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

plan/relations.go#L1183-L1184 

Added lines #L1183 - L1184 were not covered by tests

Check warning on line 1216 in plan/relations.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

plan/relations.go#L1215-L1216 

Added lines #L1215 - L1216 were not covered by tests
View more details on Codecov