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/26 16:38:16 UTC

[GitHub] [arrow-datafusion] andygrove opened a new issue, #2625: [Discuss] Move extension point traits out of core?

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

   **Is your feature request related to a problem or challenge? Please describe what you are trying to do.**
   There are two traits in the core datafusion that could perhaps be moved out into a separate crate.
   
   - VarProvider
   - FunctionRegistry
   
   Technically they could move to `datafusion-expr` since they only depend on types from that crate but that doesn't seem like the right place for them. It also doesn't seem to make sense to create a new crate just for two traits.
   
   Moving them out from core would mean:
   
   - `datafusion-proto` would no longer need to depend on the core crate
   - `SimplifyExpression` optimizer rule would have one less dependency on the core crate
   
   **Describe the solution you'd like**
   Not sure yet
   
   **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 #2625: [Discuss] Move extension point traits out of core?

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

   I think `datafusion-api` would be a good idea -- though I think we would want to try and make the design  and rationale explicit so future changes don't diverge / add new traits back into other crates


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