andygrove commented on issue #2181:
URL:
https://github.com/apache/arrow-datafusion/issues/2181#issuecomment-1100448848
So I started prototyping and ran into an issue right away. As others have
pointed out, we sometimes need to have an `Expr` that refers to a
`LogicalPlan`, such as:
``` rust
enum Expr {
Exists {
subquery: Box<LogicalPlan>
}
...
}
```
However, the LogicalPlan enum is in the `datafusion` crate which depends on
the `datafusion-expr` crate so I cannot reference `LogicalPlan` from `Expr`.
I was not paying much attention when this refactoring happened. Does it make
sense to now move `LogicalPlan` as well to `datafusion-expr` crate?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
