Improve `AccumulatorArgs` after transition to all udafs

[alamb]

          The end state in my mind now.

pub struct AccumulatorArgs<'a> {
    /// Keep, this is return type, the name might be quite confusing.
    pub data_type: &'a DataType,

    /// We might only need one of `schema` or `dfschema`. It is likely we keep `dfschema`, since we can get `schema` from it.
    pub schema: &'a Schema,

    pub dfschema: &'a DFSchema,

    /// Keep
    pub ignore_nulls: bool,

    /// Convert to physical sort exprs instead
    pub sort_exprs: &'a [Expr],

    /// Keep
    pub is_reversed: bool,

    /// We might able to get the name from expressions
    pub name: &'a str,

    /// Keep
    pub is_distinct: bool,

    /// Get the type from schema and expressions
    pub input_type: &'a DataType,

    /// Convert to physical expressions
    pub input_exprs: &'a [Expr],
}

Originally posted by @jayzhan211 in #11666 (comment)

[xinlifoobar]

take

[alamb]

FYI @jayzhan211

[jayzhan211]

There is #11739 that works on name by @lewiszlw , @xinlifoobar I think you can work on input_types

[Jefffrey]

Current state of AccumulatorArgs:

datafusion/datafusion/functions-aggregate-common/src/accumulator.rs

Lines 28 to 74 in 930620a

	#[derive(Debug, Clone)]
	pub struct AccumulatorArgs<'a> {
	/// The return field of the aggregate function.
	pub return_field: FieldRef,
	/// Input schema to the aggregate function. If you need to check data type, nullability
	/// or metadata of input arguments then you should use `expr_fields` below instead.
	pub schema: &'a Schema,
	/// Whether to ignore nulls.
	///
	/// SQL allows the user to specify `IGNORE NULLS`, for example:
	///
	/// ```sql
	/// SELECT FIRST_VALUE(column1) IGNORE NULLS FROM t;
	/// ```
	pub ignore_nulls: bool,
	/// The expressions in the `ORDER BY` clause passed to this aggregator.
	///
	/// SQL allows the user to specify the ordering of arguments to the
	/// aggregate using an `ORDER BY`. For example:
	///
	/// ```sql
	/// SELECT FIRST_VALUE(column1 ORDER BY column2) FROM t;
	/// ```
	pub order_bys: &'a [PhysicalSortExpr],
	/// Whether the aggregation is running in reverse order
	pub is_reversed: bool,
	/// The name of the aggregate expression
	pub name: &'a str,
	/// Whether the aggregate function is distinct.
	///
	/// ```sql
	/// SELECT COUNT(DISTINCT column1) FROM t;
	/// ```
	pub is_distinct: bool,
	/// The physical expression of arguments the aggregate function takes.
	pub exprs: &'a [Arc<dyn PhysicalExpr>],
	/// Fields corresponding to each expr (same order & length).
	pub expr_fields: &'a [FieldRef],
	}

I think the last remaining change before we can close this ticket is to remove schema field, though this might be difficult as mentioned here: #18100 (review)