[GitHub] [arrow-datafusion] andygrove opened a new issue, #2247: The logical plan should not depend on the physical plan

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

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

   **Is your feature request related to a problem or challenge? Please describe what you are trying to do.**
   The logical plan currently depends on the physical plan, which prevents us from moving the logical plan into its own crate alongside the logical expressions.
   
   The reason that the logical plan depends on the physical plan is that `LogicalPlan::TableScan` has a reference to a `TableProvider`.
   
   **Describe the solution you'd like**
   There is a relatively simple solution to this and that is to have `LogicalPlan::TableScan` refer to table providers by name only and then have the code retrieve the table provider from the execution context when needed.
   
   **Describe alternatives you've considered**
   None
   
   **Additional context**
   This came up when attempting to add support for subqueries.
   


-- 
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 #2247: `LogicalPlan::TableScan` should not depend on the physical plan

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

andygrove closed issue #2247: `LogicalPlan::TableScan` should not depend on the physical plan
URL: https://github.com/apache/arrow-datafusion/issues/2247


-- 
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