paleolimbot opened a new issue, #17400: URL: https://github.com/apache/datafusion/issues/17400
### Describe the bug I'm not sure if this is intentional, but this code: https://github.com/apache/datafusion/blob/d83a290d1d534f7db9849b11c39d2b0a289a62e4/datafusion/ffi/src/udaf/mod.rs#L163-L177 ...triggered a failure in one of my tests because it calls `return_type()` instead of `return_field()`. I had been making the assumption that if `return_field()` was implemented then `return_type()` shouldn't be called, so I had been returning an error from that function. I believe that the FFI schemas should be able to carry field metadata...should `return_field()` be called here? ### To Reproduce Implement an aggregate function that returns an extension type. I believe that when this happens, the metadata on output will be lost or the source `return_type()` will be called instead of the `return_field()` implementation. ### Expected behavior I would have expected the field information to propagate to the output. ### Additional context cc @timsaucer ...help! -- 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]
