[I] ExprBuilder for Physical Aggregate Expr [datafusion]

["jayzhan211 (via GitHub)" <gi...@apache.org>]

jayzhan211 opened a new issue, #11543:
URL: https://github.com/apache/datafusion/issues/11543

   ### Is your feature request related to a problem or challenge?
   
   While I working on `OrderSensitiveArrayAgg`, I found we again need to add another field `is_reversed`, indicate whether the expression is reversed or not, it is currently used specific for `OrderSensitiveArrayAgg`, but I still think we can only add it the function `create_aggregate_expr`.
   
   I wonder would it be helpful to have the expression builder to replace this function at all, I think the change could probably minimze breaking change if we are going to add / remove field inside `create_aggregate_expr` in the future
   
   ### Describe the solution you'd like
   
   _No response_
   
   ### Describe alternatives you've considered
   
   Keep `create_aggregate_expr`
   
   ### Additional context
   
   _No response_


-- 
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-unsubscribe@datafusion.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org