[GitHub] [calcite] amaliujia commented on issue #1405: [CALCITE-2772] Support varargs UDF for scalar function

[GitBox <gi...@apache.org>]

amaliujia commented on issue #1405: [CALCITE-2772] Support varargs UDF for scalar function
URL: https://github.com/apache/calcite/pull/1405#issuecomment-535739797
 
 
   @danny0405 
   
   One example I found for varargs use case is when I tried to built ZetaSQL on top of Calcite. ZetaSQL has a `concat` function definition that accepts varargs [1]. So varargs will make `concat(string...)` implementable. I will agree `concat(string...)` can be written as a nested of binary `concat`, but it makes query more readable if need to concat more than 3 strings.  
   
   
   [1] : https://github.com/google/zetasql/blob/master/docs/string_functions.md#concat 

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services