Skip to content

Move LogicalPlan to datafusion-expr crate #2245

@andygrove

Description

@andygrove

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
I am trying to implement support for the expression EXISTS (<subquery>) and this requires having a new Expr variant that refers to a LogicalPlan. However, this is not possible because LogicalPlan is in the core datafusion crate which depends on the datafusion-expr crate that contains Expr, and therefore it is not possible to reference LogicalPlan from Expr.

Describe the solution you'd like
LogicalPlan and Expr should live in the same crate IMO so I propose moving LogicalPlan into the datafusion-expr crate.

Describe alternatives you've considered
None

Additional context
See the discussion in #2181 for more context

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions