If `AggregateFunctionExpr` is not cloned massively, we could remove `Arc`, similar to that we doesn't wrap `Arc` for `Expr`. _Originally posted by @jayzhan211 in https://github.com/apache/datafusion/pull/12096#discussion_r1726365405_