Open sunheyi6 opened 1 month ago
[!IMPORTANT]
Review skipped
Auto reviews are disabled on this repository.
Please check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
The changes primarily involve updating the import paths for the LogicalPlan
from query::plan
to datafusion_expr
. This update affects several files, including repl.rs
, transform.rs
, and copy_table_to.rs
, signifying a shift in module structure or dependencies to use the datafusion_expr
crate.
File Path | Summary of Changes |
---|---|
src/cmd/src/cli/repl.rs |
Updated LogicalPlan import from datafusion_expr and adjusted its usage. |
src/flow/src/transform.rs |
Updated LogicalPlan import from datafusion_expr and modified references. |
src/operator/src/statement/... |
Updated LogicalPlan import from datafusion_expr and modified its usage. |
In the land of code, where data flows, A shift in plans, the import shows. From
query::plan
todatafusion_expr
, Our logical paths now freshly confer. With every change, the system grows, As bunnies cheer, the progress glows. 🌟🐇
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?
Thanks for submitting the PR. To remove the wrapper type, we also need to remove the definition of our LogicalPlan
and use Datafusion's LogicalPlan
directly.
#[derive(Clone, Debug)]
pub enum LogicalPlan {
DfPlan(DfLogicalPlan),
}
You can refer to https://github.com/GreptimeTeam/greptimedb/pull/3995 which removes the wrapper type for Expr
.
@sunheyi6 A conflict has to be resolved.
@sunheyi6 A conflict has to be resolved.
OK, I'll handle it right away.
Please fix the compiler errors https://github.com/GreptimeTeam/greptimedb/actions/runs/10000323455/job/27646084014?pr=4384
Please fix the compiler errors https://github.com/GreptimeTeam/greptimedb/actions/runs/10000323455/job/27646084014?pr=4384
ok,i will fix
https://github.com/GreptimeTeam/greptimedb/blob/main/CONTRIBUTING.md#before-pr Please refer to the contributing guide to run all tests and the clippy check locally first. Also please make sure your code can compile.
I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
as title, part of https://github.com/GreptimeTeam/greptimedb/issues/3884
What's changed and what's your intention?
As the title said.
Checklist
Summary by CodeRabbit
datafusion_expr::LogicalPlan
for improved code maintainability.