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

[feat/Discussion] Move polars engine affinity logic to the rust layer #21102

Open
Matt711 opened this issue Feb 5, 2025 · 0 comments
Open

[feat/Discussion] Move polars engine affinity logic to the rust layer #21102

Matt711 opened this issue Feb 5, 2025 · 0 comments
Labels
enhancement New feature or an improvement of an existing feature

Comments

@Matt711
Copy link

Matt711 commented Feb 5, 2025

Description

In #20717, we created a new polars config option to set the POLARS_ENGINE_AFFINITY. The current dispatching logic is handled in the python layer in .collect. Ie.

if get_engine_affinity() == "streaming":
    new_streaming = True

We should move this logic main optimizer loop in the rust layer or some other part of the polars rust layer (original comment). This issue is meant for discussion on possible ways to achieve this.

Also, feel free to assign this issue to me

CC @ritchie46 @wence-

@Matt711 Matt711 added the enhancement New feature or an improvement of an existing feature label Feb 5, 2025
@Matt711 Matt711 changed the title [feat/Discussion] Move polars engine affinity logic to the main optimizer loop in rust [feat/Discussion] Move polars engine affinity logic to the rust layer Feb 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or an improvement of an existing feature
Projects
None yet
Development

No branches or pull requests

1 participant