You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "alamb (via GitHub)" <gi...@apache.org> on 2024/03/09 15:54:41 UTC

[I] Move `datafusion_array_function` specific rewrite rules like to `datafusion_functions_array` crate [arrow-datafusion]

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

   ### Is your feature request related to a problem or challenge?
   
   _No response_
   
   ### Describe the solution you'd like
   
   _No response_
   
   ### Describe alternatives you've considered
   
   _No response_
   
   ### Additional context
   
   _No response_


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

Re: [I] Move `datafusion_array_function` specific rewrite rules like to `datafusion_functions_array` crate [arrow-datafusion]

Posted by "alamb (via GitHub)" <gi...@apache.org>.
alamb commented on issue #9519:
URL: https://github.com/apache/arrow-datafusion/issues/9519#issuecomment-1988327040

   > Should we change `add_analyzer_rule` to a map structure to avoid duplication similar to `register_udf`?
   
   I think the order of the rules is important. And some optimizer rules now are run multiple times 
   
   However, we may need to add some new way to add analyzer rules somewhere other than after all the other ("`push_front`?) maybe 🤔 


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

Re: [I] Move `datafusion_array_function` specific rewrite rules like to `datafusion_functions_array` crate [arrow-datafusion]

Posted by "alamb (via GitHub)" <gi...@apache.org>.
alamb closed issue #9519: Move `datafusion_array_function` specific rewrite rules like to `datafusion_functions_array` crate
URL: https://github.com/apache/arrow-datafusion/issues/9519


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

Re: [I] Move `datafusion_array_function` specific rewrite rules like to `datafusion_functions_array` crate [arrow-datafusion]

Posted by "jayzhan211 (via GitHub)" <gi...@apache.org>.
jayzhan211 commented on issue #9519:
URL: https://github.com/apache/arrow-datafusion/issues/9519#issuecomment-1988265661

   Should we change `add_analyzer_rule` to a map structure to avoid duplication similar to `register_udf`?


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