Jonah Gao

I worry that judging whether a query is non-deterministic may be not easy. Perhaps we can first leave this judgment to the user, only do this when the user specifies...

Due to parallel execution, non-deterministic behavior occurs more frequently than in traditional databases. I'm not sure if it's appropriate to disable inline CTE for all non-deterministic queries, as inline has...

It should be caused by [exprlist_to_fields_aggregate](https://github.com/apache/datafusion/blob/a0fccbf886346fde5dfbda136149ec98bbd6e952/datafusion/expr/src/utils.rs#L745). There are two columns named 'c', one from the aggregated input and the other from the output. `exprlist_to_fields_aggregate` forcibly uses the column 'c' from...

[exprlist_to_fields_aggregate](https://github.com/apache/datafusion/blob/a0fccbf886346fde5dfbda136149ec98bbd6e952/datafusion/expr/src/utils.rs#L745) was introduced by #2486, I doubt whether we still need it now because removing it won't break any tests.

take

It is caused by the [columnize_expr](https://github.com/apache/datafusion/blob/721974456742bf44bdae291b3114bc23fe478bcd/datafusion/expr/src/utils.rs#L772) function. Based on the comments, I guess `columnize_expr` is for special needs related to aggregation, and it was introduced in PR #55. Maybe we...

take

Is it possible to make the `array_slice` function accept a vector as its parameter? ```rust fn array_slice(args: Vec) { } ``` When making its UDF function, we need to do...

I submitted a formal PR #10469 to address this.

I plan to merge this PR now as it might conflict with #10646. cc @waynexia