[GitHub] [arrow-datafusion] andygrove opened a new issue, #2251: Move logical expression type coercion logic from `physical-expr` crate to `expr` crate

[GitBox <gi...@apache.org>]

andygrove opened a new issue, #2251:
URL: https://github.com/apache/arrow-datafusion/issues/2251

   **Is your feature request related to a problem or challenge? Please describe what you are trying to do.**
   We have logic to determine the return type of various logical expressions and this logic is called from `expr_schema.rs` in the `logical_plan` module but the code is located in the `physical-expr` crate and it does not need to be there. We could move this logic into the `expr` crate instead and this would remove more `LogicalPlan` dependencies on the `datafusion-physical-expr` crate.
   
   **Describe the solution you'd like**
   Refactor and move some code from `physical-expr` crate to `expr`.
   
   **Describe alternatives you've considered**
   None
   
   **Additional context**
   This work is being driven by the need to have logical plan and logical expression in the same crate so that we can implement subquery expressions.
   


-- 
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: github-unsubscribe@arrow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow-datafusion] andygrove closed issue #2251: Move logical expression type coercion logic from `physical-expr` crate to `expr` crate

[GitBox <gi...@apache.org>]

andygrove closed issue #2251: Move logical expression type coercion logic from `physical-expr` crate to `expr` crate
URL: https://github.com/apache/arrow-datafusion/issues/2251


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org