Olawoyin007 commented on PR #23415: URL: https://github.com/apache/datafusion/pull/23415#issuecomment-4927242187
Closing in favor of #23421 - the typed `encode_function<F>` / `decode_function<F>` surface with the erasure confined to the dispatch traits is clearly the better shape, and the erroring `config_options()` default fixes a real footgun in my version. Thanks for engaging with the design point by point rather than just bouncing it; the `TypeId`-routed erased channel is a trick I'll reuse. Happy to pick up one of the remaining expression ports on #22418 once the capability API lands. 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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]
