alamb commented on issue #8347: URL: https://github.com/apache/arrow-datafusion/issues/8347#issuecomment-1834584844
> @alamb I am probably losing some pre-requisite, the Expr doesn't contain a WindowUDF variant but only a WindowFunction variant https://github.com/apache/arrow-datafusion/blob/main/datafusion/expr/src/expr.rs#L156 there is an AggregateUDF though... You are right that the ticket description was incorrect, as @haohuaijin noted in https://github.com/apache/arrow-datafusion/issues/8347#issuecomment-1832226281 I have updated it to say "rename WindowFunction` to `WindowFunctionDefinition`" -- sorry for the confusion I think `AggregateUDF` is covered by https://github.com/apache/arrow-datafusion/issues/8346 -- 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]
