-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Closed
Description
Is your feature request related to a problem or challenge?
part of #14914
The output of AggregateExec also seems to contain redundant information.
┌─────────────┴─────────────┐
│ AggregateExec │
│ -------------------- │
│ aggr: count(Int64(1)) │
│ mode: Final │
└─────────────┬─────────────┘
Describe the solution you'd like
I debugged the code and found that the name of AggregateFunctionExpr is constructed in create_aggregate_expr_and_maybe_filter. In this function, debug information is generated for all Expr instances through Expr's SchemaDisplay.
To address this issue, I propose the following solution:
- Add a new member
sql_nametoAggregateFunctionExpr. - Introduce a new method
fmt_sql_name()forExpr, similar toschema_name(), and override it inAggregateFunctionto generatesql_name. - Modify
fmt_asinAggregateExecto outputaggr_expr.sql_nameinstead ofaggr_expr.name.
Describe alternatives you've considered
No response
Additional context
No response
Metadata
Metadata
Assignees
Labels
No labels