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/01/01 19:06:10 UTC

[I] Cleaning internal implementation of ScalarUDF to use `ScalarUDFImpl` (rather than the function pointers) [arrow-datafusion]

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

   ### Is your feature request related to a problem or challenge?
   
   Follow on to https://github.com/apache/arrow-datafusion/pull/8578 / https://github.com/apache/arrow-datafusion/issues/8568
   
   
   
   
   ### Describe the solution you'd like
   
   Basically now that we have a trait object based way of defining UDFs the internal function pointer implementation is awkward and unecessary. We could use the trait object directly as discussed in https://github.com/apache/arrow-datafusion/pull/8578/files#r1430521000
   
   ### 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] Clean internal implementation of ScalarUDF to use `ScalarUDFImpl` (rather than the function pointers) [arrow-datafusion]

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

   Hi @alamb, any ideas when this feature will be release? I am working on a project with a lot of need to custom UDFs and would be awesome to use the new API.  
   
   Thank you!


-- 
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] Clean internal implementation of ScalarUDF to use `ScalarUDFImpl` (rather than the function pointers) [arrow-datafusion]

Posted by "alamb (via GitHub)" <gi...@apache.org>.
alamb closed issue #8712: Clean internal implementation of ScalarUDF to use `ScalarUDFImpl` (rather than the function pointers)
URL: https://github.com/apache/arrow-datafusion/issues/8712


-- 
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] Clean internal implementation of ScalarUDF to use `ScalarUDFImpl` (rather than the function pointers) [arrow-datafusion]

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

   > Hi @alamb, any ideas when this feature will be released? I am working on a project with a lot of custom UDFs and it would be awesome to use the new API.
   > 
   > Thank you!
   
   I believe @andygrove  plans to do so soon -- https://github.com/apache/arrow-datafusion/issues/8863 tracks the release that will have this feature


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