Skip to content

perf: Add dynamic predicates for TopK - #26495

Merged
ritchie46 merged 15 commits into
mainfrom
dynamic_pred
Feb 20, 2026
Merged

ritchie46 merged 15 commits into
mainfrom
dynamic_pred

Conversation

@ritchie46

Copy link
Copy Markdown
Member

No description provided.

@github-actions

Copy link
Copy Markdown
Contributor

The uncompressed lib size after this PR is 52.6520 MB.

@github-actions

Copy link
Copy Markdown
Contributor

The uncompressed lib size after this PR is 52.9606 MB.

@github-actions

Copy link
Copy Markdown
Contributor

The uncompressed lib size after this PR is 53.2151 MB.

@orlp orlp changed the title WIP: Dynamic predicate for TopK perf: Add dynamic predicates for TopK Feb 17, 2026
@github-actions github-actions Bot added performance Performance issues or improvements python Related to Python Polars rust Related to Rust Polars and removed title needs formatting labels Feb 17, 2026
@github-actions

Copy link
Copy Markdown
Contributor

The uncompressed lib size after this PR is 53.2149 MB.

@codecov

codecov Bot commented Feb 17, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 84.65347% with 31 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.72%. Comparing base (7d83b12) to head (c0e1329).
⚠️ Report is 34 commits behind head on main.

Files with missing lines Patch % Lines
crates/polars-stream/src/nodes/top_k.rs 83.65% 17 Missing ⚠️
.../src/plans/optimizer/predicate_pushdown/dynamic.rs 89.47% 6 Missing ⚠️
...s/polars-plan/src/plans/aexpr/function_expr/mod.rs 40.00% 3 Missing ⚠️
crates/polars-plan/src/plans/builder_ir.rs 0.00% 1 Missing ⚠️
crates/polars-plan/src/plans/ir/format.rs 75.00% 1 Missing ⚠️
crates/polars-plan/src/plans/optimizer/mod.rs 88.88% 1 Missing ⚠️
.../polars-python/src/lazyframe/visitor/expr_nodes.rs 0.00% 1 Missing ⚠️
...rates/polars-python/src/lazyframe/visitor/nodes.rs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #26495      +/-   ##
==========================================
- Coverage   81.17%   78.72%   -2.45%     
==========================================
  Files        1792     1795       +3     
  Lines      244511   245195     +684     
  Branches     3078     3080       +2     
==========================================
- Hits       198475   193041    -5434     
- Misses      45250    51368    +6118     
  Partials      786      786              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions

Copy link
Copy Markdown
Contributor

The uncompressed lib size after this PR is 53.8417 MB.

@ritchie46
ritchie46 merged commit b960ca3 into main Feb 20, 2026
29 checks passed
@ritchie46
ritchie46 deleted the dynamic_pred branch February 20, 2026 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

performance Performance issues or improvements python Related to Python Polars rust Related to Rust Polars

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants