alamb opened a new issue, #11804: URL: https://github.com/apache/datafusion/issues/11804
### Is your feature request related to a problem or challenge? As @jayzhan211 points out on https://github.com/apache/datafusion/pull/11013#discussion_r1702704630 the `ordering_req` field is not supported when serializating/deserializing aggregate udfs ### Describe the solution you'd like Add `ordering_req` to the serialization / deserializton of aggregateudf and add a test ### Describe alternatives you've considered Here is the code where the ordeirng is ignored when serializing https://github.com/apache/datafusion/blob/f4e519f9df9ab5972638d3f2743da01887a52668/datafusion/proto/src/physical_plan/mod.rs#L480-L481 ### Additional context I think technically this may be a regression as builtin functions had proper `ordering` serialization -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
