jorgecarleitao commented on issue #305: URL: https://github.com/apache/arrow-datafusion/issues/305#issuecomment-836456326
They are independent solutions: In the former, we have a new `PhysicalExpr`, that executes evaluates in a different way (by passing the length to the function) In the latter, the planner would query the number of arguments of the expression, and if it is zero, would pass a `NullArray` to it (I do not have the exact solution atm). In my opinion this issue should be written in a generic form, so that it focus on the problem and not on the solution. E.g. instead of "Allow ScalarFunctionImplementation to...", use "Allow execution of zero-argument functions". It is not obvious to me that changing the signature of `ScalarFunctionImplementation` is the right approach here. -- 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: us...@infra.apache.org
