-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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: move describe to IR instead of DSL #16191
Conversation
58b7fe3
to
bf45fb7
Compare
There are quite some tests that are failing. I have some trouble running test suite locally, will try to get it to run. |
bf45fb7
to
110fa35
Compare
110fa35
to
df3676b
Compare
df3676b
to
c7a6072
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #16191 +/- ##
==========================================
- Coverage 80.99% 80.99% -0.01%
==========================================
Files 1392 1393 +1
Lines 178920 179440 +520
Branches 2892 2907 +15
==========================================
+ Hits 144923 145333 +410
- Misses 33500 33601 +101
- Partials 497 506 +9 ☔ View full report in Codecov by Sentry. |
This PR moves the
LazyFrame::describe_optimized_plan
to using theIR
instead ofDslPlan
. Most code is based on the old formatting code and is located in thepolars-plan/src/logical-plan/alp/format.rs
file.