Closed waynexia closed 5 days ago
The recent changes encompass updating the Cargo.toml
file to switch the repository URLs for multiple datafusion
dependencies from the Apache repository to waynexia/arrow-datafusion
. Additionally, the datafusion.rs
and planner.rs
files in the src/query/src/dist_plan
directory have been modified to enhance the optimization processes for logical plans and to update the handling of Explain
plans and MergeScan
.
File | Change Summary |
---|---|
Cargo.toml |
Changed repository URLs for datafusion and related dependencies to waynexia/arrow-datafusion . |
src/query/src/datafusion.rs |
Added MergeScanLogicalPlan import and modified handling of Explain plans and logical plan optimization. |
src/query/src/dist_plan/planner.rs |
Refactored optimize_input_logical_plan to optimize directly without explicit Analyzer and updated create_physical_plan . |
sequenceDiagram
participant Client
participant DistExtensionPlanner
participant SessionState
participant LogicalPlan
participant PhysicalPlan
Client->>DistExtensionPlanner: Call optimize_input_logical_plan
DistExtensionPlanner->>SessionState: Use session_state
DistExtensionPlanner->>LogicalPlan: Optimize input logical plan
LogicalPlan-->>DistExtensionPlanner: Return optimized plan
Client->>DistExtensionPlanner: Call create_physical_plan
DistExtensionPlanner->>OptimizedPlan: Use optimized plan
OptimizedPlan-->>PhysicalPlan: Create physical plan
PhysicalPlan-->>DistExtensionPlanner: Return physical plan
DistExtensionPlanner-->>Client: Return result
In the code, new paths align,
From Apache to Wayne's design,
Optimized with careful thread,
Logical plans now forge ahead.
Arrows in the data's flight,
Merge and scan, our plans delight! 🚀🐇
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Attention: Patch coverage is 80.64516%
with 6 lines
in your changes missing coverage. Please review.
Project coverage is 84.91%. Comparing base (
da0c840
) to head (adb102f
). Report is 5 commits behind head on main.
I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
What's changed and what's your intention?
!!! DO NOT LEAVE THIS BLOCK EMPTY !!!
Please explain IN DETAIL what the changes are in this PR and why they are needed:
Checklist
Summary by CodeRabbit
Updates
Enhancements