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.

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

refactor(polars): update the polars backend to use the new relational abstractions #7868

Merged
merged 1 commit into from
Jan 4, 2024

Conversation

kszucs
Copy link
Member

@kszucs kszucs commented Dec 30, 2023

Bare minimum changes required for the polars backend to work with the new relational operations. Since polars' join API follows the same semantics as pandas, I'm using the pandas specific rewrites here.

In the future we may want to rewrite the compiler similarly to the pandas one: using node.map() and Dispatched.

@cpcloud cpcloud added refactor Issues or PRs related to refactoring the codebase polars The polars backend labels Dec 30, 2023
@kszucs kszucs force-pushed the tes-polars branch 3 times, most recently from 4b1be6a to 6cb3f66 Compare December 30, 2023 11:35
Copy link
Member

@cpcloud cpcloud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ship it!

@cpcloud cpcloud enabled auto-merge (squash) January 4, 2024 16:14
@cpcloud cpcloud merged commit f521a07 into ibis-project:the-epic-split Jan 4, 2024
31 checks passed
cpcloud pushed a commit that referenced this pull request Jan 5, 2024
… abstractions (#7868)

Bare minimum changes required for the polars backend to work with the
new relational operations. Since polars' join API follows the same
semantics as pandas, I'm using the pandas specific rewrites here.

In the future we may want to rewrite the compiler similarly to the
pandas one: using `node.map()` and `Dispatched`.
cpcloud pushed a commit that referenced this pull request Jan 12, 2024
… abstractions (#7868)

Bare minimum changes required for the polars backend to work with the
new relational operations. Since polars' join API follows the same
semantics as pandas, I'm using the pandas specific rewrites here.

In the future we may want to rewrite the compiler similarly to the
pandas one: using `node.map()` and `Dispatched`.
cpcloud pushed a commit that referenced this pull request Jan 13, 2024
… abstractions (#7868)

Bare minimum changes required for the polars backend to work with the
new relational operations. Since polars' join API follows the same
semantics as pandas, I'm using the pandas specific rewrites here.

In the future we may want to rewrite the compiler similarly to the
pandas one: using `node.map()` and `Dispatched`.
cpcloud pushed a commit that referenced this pull request Jan 17, 2024
… abstractions (#7868)

Bare minimum changes required for the polars backend to work with the
new relational operations. Since polars' join API follows the same
semantics as pandas, I'm using the pandas specific rewrites here.

In the future we may want to rewrite the compiler similarly to the
pandas one: using `node.map()` and `Dispatched`.
kszucs added a commit to kszucs/ibis that referenced this pull request Feb 1, 2024
… abstractions (ibis-project#7868)

Bare minimum changes required for the polars backend to work with the
new relational operations. Since polars' join API follows the same
semantics as pandas, I'm using the pandas specific rewrites here.

In the future we may want to rewrite the compiler similarly to the
pandas one: using `node.map()` and `Dispatched`.
kszucs added a commit to kszucs/ibis that referenced this pull request Feb 1, 2024
… abstractions (ibis-project#7868)

Bare minimum changes required for the polars backend to work with the
new relational operations. Since polars' join API follows the same
semantics as pandas, I'm using the pandas specific rewrites here.

In the future we may want to rewrite the compiler similarly to the
pandas one: using `node.map()` and `Dispatched`.
kszucs added a commit to kszucs/ibis that referenced this pull request Feb 1, 2024
… abstractions (ibis-project#7868)

Bare minimum changes required for the polars backend to work with the
new relational operations. Since polars' join API follows the same
semantics as pandas, I'm using the pandas specific rewrites here.

In the future we may want to rewrite the compiler similarly to the
pandas one: using `node.map()` and `Dispatched`.
kszucs added a commit to kszucs/ibis that referenced this pull request Feb 2, 2024
… abstractions (ibis-project#7868)

Bare minimum changes required for the polars backend to work with the
new relational operations. Since polars' join API follows the same
semantics as pandas, I'm using the pandas specific rewrites here.

In the future we may want to rewrite the compiler similarly to the
pandas one: using `node.map()` and `Dispatched`.
kszucs added a commit to kszucs/ibis that referenced this pull request Feb 2, 2024
… abstractions (ibis-project#7868)

Bare minimum changes required for the polars backend to work with the
new relational operations. Since polars' join API follows the same
semantics as pandas, I'm using the pandas specific rewrites here.

In the future we may want to rewrite the compiler similarly to the
pandas one: using `node.map()` and `Dispatched`.
kszucs added a commit to kszucs/ibis that referenced this pull request Feb 2, 2024
… abstractions (ibis-project#7868)

Bare minimum changes required for the polars backend to work with the
new relational operations. Since polars' join API follows the same
semantics as pandas, I'm using the pandas specific rewrites here.

In the future we may want to rewrite the compiler similarly to the
pandas one: using `node.map()` and `Dispatched`.
cpcloud pushed a commit to cpcloud/ibis that referenced this pull request Feb 4, 2024
… abstractions (ibis-project#7868)

Bare minimum changes required for the polars backend to work with the
new relational operations. Since polars' join API follows the same
semantics as pandas, I'm using the pandas specific rewrites here.

In the future we may want to rewrite the compiler similarly to the
pandas one: using `node.map()` and `Dispatched`.
cpcloud pushed a commit to cpcloud/ibis that referenced this pull request Feb 5, 2024
… abstractions (ibis-project#7868)

Bare minimum changes required for the polars backend to work with the
new relational operations. Since polars' join API follows the same
semantics as pandas, I'm using the pandas specific rewrites here.

In the future we may want to rewrite the compiler similarly to the
pandas one: using `node.map()` and `Dispatched`.
kszucs added a commit that referenced this pull request Feb 5, 2024
… abstractions (#7868)

Bare minimum changes required for the polars backend to work with the
new relational operations. Since polars' join API follows the same
semantics as pandas, I'm using the pandas specific rewrites here.

In the future we may want to rewrite the compiler similarly to the
pandas one: using `node.map()` and `Dispatched`.
kszucs added a commit that referenced this pull request Feb 6, 2024
… abstractions (#7868)

Bare minimum changes required for the polars backend to work with the
new relational operations. Since polars' join API follows the same
semantics as pandas, I'm using the pandas specific rewrites here.

In the future we may want to rewrite the compiler similarly to the
pandas one: using `node.map()` and `Dispatched`.
kszucs added a commit that referenced this pull request Feb 6, 2024
… abstractions (#7868)

Bare minimum changes required for the polars backend to work with the
new relational operations. Since polars' join API follows the same
semantics as pandas, I'm using the pandas specific rewrites here.

In the future we may want to rewrite the compiler similarly to the
pandas one: using `node.map()` and `Dispatched`.
cpcloud pushed a commit to cpcloud/ibis that referenced this pull request Feb 12, 2024
… abstractions (ibis-project#7868)

Bare minimum changes required for the polars backend to work with the
new relational operations. Since polars' join API follows the same
semantics as pandas, I'm using the pandas specific rewrites here.

In the future we may want to rewrite the compiler similarly to the
pandas one: using `node.map()` and `Dispatched`.
cpcloud pushed a commit that referenced this pull request Feb 12, 2024
… abstractions (#7868)

Bare minimum changes required for the polars backend to work with the
new relational operations. Since polars' join API follows the same
semantics as pandas, I'm using the pandas specific rewrites here.

In the future we may want to rewrite the compiler similarly to the
pandas one: using `node.map()` and `Dispatched`.
cpcloud pushed a commit to cpcloud/ibis that referenced this pull request Feb 12, 2024
… abstractions (ibis-project#7868)

Bare minimum changes required for the polars backend to work with the
new relational operations. Since polars' join API follows the same
semantics as pandas, I'm using the pandas specific rewrites here.

In the future we may want to rewrite the compiler similarly to the
pandas one: using `node.map()` and `Dispatched`.
cpcloud pushed a commit that referenced this pull request Feb 12, 2024
… abstractions (#7868)

Bare minimum changes required for the polars backend to work with the
new relational operations. Since polars' join API follows the same
semantics as pandas, I'm using the pandas specific rewrites here.

In the future we may want to rewrite the compiler similarly to the
pandas one: using `node.map()` and `Dispatched`.
kszucs added a commit that referenced this pull request Feb 12, 2024
… abstractions (#7868)

Bare minimum changes required for the polars backend to work with the
new relational operations. Since polars' join API follows the same
semantics as pandas, I'm using the pandas specific rewrites here.

In the future we may want to rewrite the compiler similarly to the
pandas one: using `node.map()` and `Dispatched`.
ncclementi pushed a commit to ncclementi/ibis that referenced this pull request Feb 21, 2024
… abstractions (ibis-project#7868)

Bare minimum changes required for the polars backend to work with the
new relational operations. Since polars' join API follows the same
semantics as pandas, I'm using the pandas specific rewrites here.

In the future we may want to rewrite the compiler similarly to the
pandas one: using `node.map()` and `Dispatched`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
polars The polars backend refactor Issues or PRs related to refactoring the codebase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants