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

feat(connect): `with_columns_renamed` #3386

Merged

andrewgazelka merged 2 commits into main from andrew/connect-with-columns-renamed

Dec 19, 2024

Merged

feat(connect): `with_columns_renamed` #3386

slight restructure

Codecov / codecov/patch succeeded Dec 19, 2024 in 0s

80.00% of diff hit (target 77.84%)

View this Pull Request on Codecov

80.00% of diff hit (target 77.84%)

Annotations

Check warning on line 116 in src/daft-connect/src/translation/logical_plan.rs

codecov / codecov/patch

src/daft-connect/src/translation/logical_plan.rs#L116

Added line #L116 was not covered by tests

Check warning on line 19 in src/daft-connect/src/translation/logical_plan/with_columns_renamed.rs

codecov / codecov/patch

src/daft-connect/src/translation/logical_plan/with_columns_renamed.rs#L19

Added line #L19 was not covered by tests

Check warning on line 38 in src/daft-connect/src/translation/logical_plan/with_columns_renamed.rs

codecov / codecov/patch

src/daft-connect/src/translation/logical_plan/with_columns_renamed.rs#L35-L38

Added lines #L35 - L38 were not covered by tests

View more details on Codecov