andygrove commented on issue #2644:
URL: 
https://github.com/apache/arrow-datafusion/issues/2644#issuecomment-1145943937

   > The part that's not straightforward is the corresponding 
`TableSource::as_provider()`, as it creates a circular dependency between the 
crates.
   
   Yes, the reason for introducing `TableSource` in the first place was to stop 
logical plans depending on execution plans (via the `TableProvider::scan` 
method) and to allow us to move logical plans and expressions out of the core 
crate. It is intentionally difficult to reintroduce that dependency.


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

Reply via email to