Jimexist commented on a change in pull request #328:
URL: https://github.com/apache/arrow-datafusion/pull/328#discussion_r633031732



##########
File path: datafusion/src/physical_plan/functions.rs
##########
@@ -207,6 +215,14 @@ pub enum BuiltinScalarFunction {
     RegexpMatch,
 }
 
+impl BuiltinScalarFunction {
+    /// an allowlist of functions to take zero arguments, so that they will 
get special treatment
+    /// while executing.
+    fn supports_zero_argument(&self) -> bool {

Review comment:
       had to add this whitelist because vararg funcs like `array` will not 
accept 0 args




-- 
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:
[email protected]


Reply via email to