rename

Signed-off-by: jayzhan211 <jayzhan211@gmail.com>

Author: jayzhan211

datafusion/core/src/physical_planner.rs

@@ -276,7 +276,7 @@ fn create_physical_name(e: &Expr, is_first_expr: bool) -> Result<String> {
                     .iter()
                     .map(|e| create_physical_name(e, false))
                     .collect::<Result<Vec<_>>>()?;
-                Ok(format!("{}({})", fun.name(), names.join(",")))
+                Ok(format!("{}({})", fun.name(), names.join(", ")))
             }
             AggregateFunctionDefinition::Name(_) => {
                 internal_err!("Aggregate function `Expr` with name should be resolved.")

datafusion/expr/src/expr.rs

@@ -1853,7 +1853,7 @@ fn create_name(e: &Expr) -> Result<String> {
                 AggregateFunctionDefinition::UDF(..) => {
                     let names: Vec<String> =
                         args.iter().map(create_name).collect::<Result<_>>()?;
-                    names.join(",")
+                    names.join(", ")
                 }
             };
             let mut info = String::new();