berkaysynnada commented on issue #15111:
URL: https://github.com/apache/datafusion/issues/15111#issuecomment-2726959050

   > physical-plan is not possible to import `datasource`, you would end up 
moving everything inside physical-plan.
   
   Why are you thinking so? If you worry about the catalog, when we have 
physical-plan-common, we can easily convert physical-plan dependency of catalog 
crate to physical-plan-common.
   
   ```mermaid
   flowchart TD
       physical-plan["physical-plan"] --> 
physical-plan-common["physical-plan-common"] & datasource["datasource"]
       datasource --> physical-plan-common
       n1["catalog"] --> physical-plan-common
       datasource --> n1
   
       n1@{ shape: rect}
   ```
   
   


-- 
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-unsubscr...@datafusion.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to