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_name
toAggregateFunctionExpr
. - Introduce a new method
fmt_sql_name()
forExpr
, similar toschema_name()
, and override it inAggregateFunction
to generatesql_name
. - Modify
fmt_as
inAggregateExec
to outputaggr_expr.sql_name
instead ofaggr_expr.name
.
Describe alternatives you've considered
No response
Additional context
No response
Metadata
Metadata
Assignees
Labels
No labels