Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

perf: Re-enable common subplan elim for new-streaming engine #20443

Merged
merged 3 commits into from
Dec 25, 2024

Conversation

orlp
Copy link
Collaborator

@orlp orlp commented Dec 24, 2024

No description provided.

@github-actions github-actions bot added performance Performance issues or improvements python Related to Python Polars rust Related to Rust Polars labels Dec 24, 2024
@orlp orlp force-pushed the comm-subplan-elim branch from 120f5d5 to b0206db Compare December 24, 2024 15:36
Copy link

codecov bot commented Dec 24, 2024

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 79.00%. Comparing base (d27f7b2) to head (b0206db).
Report is 24 commits behind head on main.

Files with missing lines Patch % Lines
crates/polars-stream/src/physical_plan/mod.rs 0.00% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main   #20443   +/-   ##
=======================================
  Coverage   79.00%   79.00%           
=======================================
  Files        1563     1563           
  Lines      220338   220336    -2     
  Branches     2488     2488           
=======================================
+ Hits       174068   174073    +5     
+ Misses      45697    45690    -7     
  Partials      573      573           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@@ -86,6 +86,7 @@ pub fn lower_ir(
phys_sm: &mut SlotMap<PhysNodeKey, PhysNode>,
schema_cache: &mut PlHashMap<Node, Arc<Schema>>,
expr_cache: &mut ExprCache,
cache_nodes: &mut PlHashMap<usize, PhysNodeKey>,
Copy link
Member

Choose a reason for hiding this comment

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

That was easy. :)

@ritchie46 ritchie46 merged commit ffc5538 into pola-rs:main Dec 25, 2024
26 checks passed
@c-peters c-peters added the accepted Ready for implementation label Dec 29, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
accepted Ready for implementation performance Performance issues or improvements python Related to Python Polars rust Related to Rust Polars
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants