jayzhan211 commented on code in PR #10525: URL: https://github.com/apache/datafusion/pull/10525#discussion_r1602371415
########## datafusion/expr/src/function.rs: ########## @@ -66,22 +69,39 @@ pub struct AccumulatorArgs<'a> { /// /// If no `ORDER BY` is specified, `sort_exprs`` will be empty. pub sort_exprs: &'a [Expr], + + /// Whether the aggregate function is distinct. + pub is_distinct: bool, + + /// The input type of the aggregate function. + pub input_type: &'a DataType, +} + +/// [`GroupsAccumulatorSupportedArgs`] contains information to determine if an +/// aggregate function supports the groups accumulator. +pub struct GroupsAccumulatorSupportedArgs { Review Comment: We could do this. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org