[GitHub] [arrow-datafusion] comphead commented on issue #5600: [DISCUSSION] Add separate crate to cover spark builtin functions

["comphead (via GitHub)" <gi...@apache.org>]

comphead commented on issue #5600:
URL: https://github.com/apache/arrow-datafusion/issues/5600#issuecomment-1470977590

   @alamb you are right, we can't provide optional builtin functions, that would be more than unexpected. 
   
   In wonderful example #5568 (Implement to_unixtime function)  the same functionality can be achieved differently in  
   Spark `to_unix_timestamp(now())`
   PG `extract(epoch from now())`
   Trino `select to_unixtime(now())`
   
   I'm not even sure if we need extra crates now. My personal feeling is Trino trying to adopt all possible syntaxes. 
   Perhaps we go that way and just keep extending the list of builtin functions and in case of conflicts(if any) the PG signature/return type could be as dominant
   


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