alamb commented on a change in pull request #9600:
URL: https://github.com/apache/arrow/pull/9600#discussion_r588872840
##########
File path: rust/datafusion/src/logical_plan/expr.rs
##########
@@ -160,20 +160,26 @@ pub enum Expr {
},
/// Represents the call of a built-in scalar function with a set of
arguments.
ScalarFunction {
+ /// The input name of the function
+ input_name: String,
Review comment:
FWIW here is what postgres does for the same query (uses the lowercase,
canonical function name)
```
alamb=# select mD5('a');
md5
----------------------------------
0cc175b9c0f1b6a831c399e269772661
(1 row)
```
----------------------------------------------------------------
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
