You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/05/23 13:33:21 UTC

[GitHub] [arrow-datafusion] andygrove opened a new issue, #2599: Move logical optimizer rules out of the core datafusion crate

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

   **Is your feature request related to a problem or challenge? Please describe what you are trying to do.**
   As a user of DataFusion for SQL query planning, I would like to be able to use the logical plan optimizer rules without depending on the full datafusion crate containing the execution engine.
   
   **Describe the solution you'd like**
   I think there are two choices here:
   
   1. Move the optimizer rules up to the `datafusion-expr` crate which contains the logical plans and expressions and already contains some of the expression rewriting logic that the optimizers depend on. This option would require us to make some changes to remove dependencies on the `physical-expr` crate as noted in https://github.com/apache/arrow-datafusion/issues/2535
   2. Create a new `datafusion-optimizer` crate. This is an easier step because we can have it depend on `datafusion-physical-expr` for now and fix that later.
   
   **Describe alternatives you've considered**
   None
   
   **Additional context**
   None
   


-- 
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] alamb commented on issue #2599: Move logical optimizer rules out of the core datafusion crate

Posted by GitBox <gi...@apache.org>.
alamb commented on issue #2599:
URL: https://github.com/apache/arrow-datafusion/issues/2599#issuecomment-1135927717

   > Create a new datafusion-optimizer crate. 
   
   I would personally prefer this option as well


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

[GitHub] [arrow-datafusion] andygrove commented on issue #2599: Move logical optimizer rules out of the core datafusion crate

Posted by GitBox <gi...@apache.org>.
andygrove commented on issue #2599:
URL: https://github.com/apache/arrow-datafusion/issues/2599#issuecomment-1134688270

   @alamb What would your preference be here?


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

[GitHub] [arrow-datafusion] andygrove closed issue #2599: Move logical optimizer rules out of the core datafusion crate

Posted by GitBox <gi...@apache.org>.
andygrove closed issue #2599: Move logical optimizer rules out of the core datafusion crate
URL: https://github.com/apache/arrow-datafusion/issues/2599


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